[sqlalchemy] Re: Some open tickets

2007-02-13 Thread Paul Johnston
Hi, Ticket #422: Oh dear! I seem to have made a bit of a hash of this. Most of your comments relate to patch 1, which has actually already been committed. The other controversial bit was the change to ClauseList http://www.sqlalchemy.org/trac/wiki/ClauseList.compare. What I've done is roll

[sqlalchemy] Re: Some open tickets

2007-02-11 Thread Michael Bayer
yeah im not cool changing the compare method on ClauseList, thats a very deep place to change something which should not matter for any particular database dialect, youd have to show me what thats accomplishing. the test is more strict as an ordered test and theres been no issue with that as of