search

LauJensen / clojureql

ClojureQL is superior SQL integration for Clojure
https://clojureql.sabrecms.com
Eclipse Public License 1.0
284 stars 39 forks source link

Remove alias from GROUP BY. #96

Closed ck closed 13 years ago

ck commented 13 years ago

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

bendlas commented 13 years ago

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

cheers