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

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

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

 * status:  new => closed
 * milestone:  0.8.xx => 0.9.0
 * resolution:   => fixed
 * status_field:  in queue => completed/closed


Comment:

 r5f0b864ad02409cf668fa7db in 0.9 makes both include/exclude arguments
 mandatory, the Dialect.reflecttable method is not even needed at this
 point, though I'm leaving it in as an alternative for 3rd party dialects
 in rare circumstances.

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