[Patch] [Jython] fix topological sort

Issue #1380 resolved
Former user created an issue

Jython's goofy object ids exposes some goofyness in topological._sort. r3720 changed it to avoid hashing things:

http://www.sqlalchemy.org/trac/changeset/3720#file9

r3887 reverted some of that change (but not all)

http://www.sqlalchemy.org/trac/changeset/3887#file14

Currently it's initially assuming id(item) is the item. Attached patch has it use id(item) everywhere like r3720 does. Against 0.6 branch

-pjenvey

Comments (4)

  1. Mike Bayer repo owner
    • changed milestone to 0.6.0

    surprising, in that our non-hashable tests don't uncover this. sure this can be committed.

  2. Log in to comment