thats fixed in r2b66b5abf755, will be in 0.7.4 or you can get the tip off the download page.
On Dec 1, 2011, at 1:47 PM, Michael Bayer wrote: > nevermind, I need to randomize that list in order to trigger it > > > On Dec 1, 2011, at 1:42 PM, Michael Bayer wrote: > >> I'm not able to reproduce that, and also this code should likely be replaced >> by our existing topological sort code. Can you provide a succinct >> reproducing example ? >> >> >> On Dec 1, 2011, at 2:22 AM, Robert Forkel wrote: >> >>> Hi, >>> trying to use Inspector.get_table_names with order_by='foreign_key' >>> causes the following exception: >>> >>> Traceback (most recent call last): >>> File "db_inspector.py", line 20, in <module> >>> for table in insp.get_table_names(schema=schema, >>> order_by='foreign_key'): >>> File "lib/python2.6/site-packages/sqlalchemy/engine/reflection.py", >>> line 173, in get_table_names >>> ordered_tnames.index(ref_pos, tname) >>> TypeError: slice indices must be integers or None or have an __index__ >>> method >>> >>> which can be remdied by the following patch: >>> >>> (gulpenv)$ diff -crB lib/python2.6/site-packages/sqlalchemy/engine/ >>> reflection.py lib/python2.6/site-packages/sqlalchemy/engine/ >>> reflection.py.orig*** lib/python2.6/site-packages/sqlalchemy/engine/ >>> reflection.py 2011-12-01 08:15:01.600838080 +0100 >>> --- lib/python2.6/site-packages/sqlalchemy/engine/reflection.py.orig >>> 2011-12-01 08:14:40.980828074 +0100 >>> *************** >>> *** 169,175 **** >>> if table_pos > ref_pos: >>> ordered_tnames.pop(table_pos) # rtable >>> moves up 1 >>> # insert just below rtable >>> ! ordered_tnames.insert(ref_pos, tname) >>> tnames = ordered_tnames >>> return tnames >>> >>> --- 169,175 ---- >>> if table_pos > ref_pos: >>> ordered_tnames.pop(table_pos) # rtable >>> moves up 1 >>> # insert just below rtable >>> ! ordered_tnames.index(ref_pos, tname) >>> tnames = ordered_tnames >>> return tnames >>> >>> best regards >>> robert >>> >>> -- >>> You received this message because you are subscribed to the Google Groups >>> "sqlalchemy" group. >>> To post to this group, send email to [email protected]. >>> 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. >>> >> >> -- >> You received this message because you are subscribed to the Google Groups >> "sqlalchemy" group. >> To post to this group, send email to [email protected]. >> 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. >> > > -- > You received this message because you are subscribed to the Google Groups > "sqlalchemy" group. > To post to this group, send email to [email protected]. > 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. > -- You received this message because you are subscribed to the Google Groups "sqlalchemy" group. To post to this group, send email to [email protected]. 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.
