Re: [Sqlalchemy-tickets] [sqlalchemy] #2748: fix default argument of None for DefaultDialect.reflecttable->exclude_columns, should be () (was: reflecttable not checking for None exclude_columns)

[sqlalchemy <mi...@zz...>]

#2748: fix default argument of None for
DefaultDialect.reflecttable->exclude_columns, should be ()
------------------------------+-----------------------------------
      Reporter:  rstuart4133  |      Owner:  zzzeek
          Type:  defect       |     Status:  new
      Priority:  medium       |  Milestone:  0.8.xx
     Component:  schema       |   Severity:  trivial - <10 minutes
    Resolution:               |   Keywords:
Progress State:  in queue     |
------------------------------+-----------------------------------
Changes (by zzzeek):

 * status_field:  needs questions answered => in queue
 * severity:  minor - half an hour => trivial - <10 minutes


Comment:

 OK engine.reflecttable() was removed in 0.8 and as you can see in 0.7 it's
 marked deprecated.  So while the signature of DefaultDialect.reflecttable
 should be fixed, I'm not seeing any current public API issues.   As for
 0.7 I can check in a fix but I don't know that more 0.7 releases are
 planned at this time.

--
Ticket URL: <http://www.sqlalchemy.org/trac/ticket/2748#comment:3>
sqlalchemy <http://www.sqlalchemy.org/>
The Database Toolkit for Python