Remove alias from GROUP BY. #96

Merged
merged 1 commit into from Aug 25, 2011

2 participants

@ck
ck commented Jun 29, 2011

Dear ClojureQL team,

This fixes aliases in GROUP BY (see associated tests).

As noted, this is more of a kludge right now, since the function has following todo We shouldn't call to-fieldlist here, first in the compiler.

I am more then happy to take a stab at re-factoring the to-fieldlist, but would like to talk to someone beforehand (via irc) before I go off in the wrong direction and wasting everyones time.

Regards,

Christian

@ck ck Remove alias from GROUP BY.
This is more of a kludge right now, since the function has following todo 'We shouldn't call to-fieldlist here, first in the compiler', but I am not sure what exactly Lau has in mind.
8ec9109
@bendlas bendlas merged commit 995590e into LauJensen:master Aug 25, 2011
@bendlas
Collaborator

Thanks a lot!
If you want to do some more refactoring, just tell me what you need to know.

cheers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment