Re: [Sqlalchemy-tickets] [sqlalchemy] #2748: reflecttable not checking for None exclude_columns

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

#2748: reflecttable not checking for None exclude_columns
-------------------------------------------+-------------------------------
      Reporter:  rstuart4133               |      Owner:  zzzeek
          Type:  defect                    |     Status:  new
      Priority:  medium                    |  Milestone:  0.8.xx
     Component:  schema                    |   Severity:  minor - half an
    Resolution:                            |  hour
Progress State:  needs questions answered  |   Keywords:
-------------------------------------------+-------------------------------
Changes (by zzzeek):

 * status_field:  awaiting triage => needs questions answered
 * component:  cextensions => schema
 * severity:  no triage selected yet => minor - half an hour
 * milestone:   => 0.8.xx


Comment:

 `exclude_columns` has not been considered public API, even though it seems
 to have found its way to reflection.reflecttable, it's not present in the
 usual `Table()` API.  Are you using it directly or is there some codepath
 that it becomes `None`? (I'm not seeing one).   It's assumed to always be
 non-None right now (the boolean checks are for emptiness).

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