Michael Bayer wrote:
> 
> I added distinct() to selectresults as a method and made the 
> unit test a little clearer (since i dont like relying on the 
> selectresults "mod")...
> 
>         q = sess.query(Department)
>         d = SelectResults(q)
>         d =
> d.join_to('employees').filter(Employee.c.name.startswith('J'))
>         d = d.distinct()
>         d = d.order_by([desc(Department.c.name)])
> 
> 

...and...

> 
> for the order by getting removed during the select, that 
> seemed to be an optimization that got stuck in there and 
> since this is a really fringe use case its never come up, so 
> i removed it and added your test case (only with 
> distinct=True) in rev 2301.
> 

I think you're slipping - I had to wait a whole three and a half hours
for this fix ;-) Seriously, thanks again,

Simon

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"sqlalchemy" group.
To post to this group, send email to sqlalchemy@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/sqlalchemy?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to