Re: [Sqlalchemy-tickets] [sqlalchemy] #2778: per-attribute callables due to options, possible reorg to remove performance overhead

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

#2778: per-attribute callables due to options, possible reorg to remove
performance overhead
-----------------------------------+----------------------------------
      Reporter:  zzzeek            |      Owner:  zzzeek
          Type:  defect            |     Status:  closed
      Priority:  medium            |  Milestone:  0.8.xx
     Component:  orm               |   Severity:  minor - half an hour
    Resolution:  fixed             |   Keywords:
Progress State:  completed/closed  |
-----------------------------------+----------------------------------
Changes (by zzzeek):

 * status:  new => closed
 * milestone:  0.9.0 => 0.8.xx
 * resolution:   => fixed
 * severity:  very major - up to 2 days => minor - half an hour
 * status_field:  in queue => completed/closed


Comment:

 So with some profiling added, previously defer() would bump the callcounts
 up significantly, it now uses fewer calls than that of loading the column.
 the change here isn't that dramatic, just not bundling the "state" with
 those callables (so we only need create one per col, not per col per row)
 and producing the closure after retrieving the `AttributeImpl`, not
 before.

 r731a4daf63dc0fdb784d195e89c5f357420657fb 0.8

 r5b65af098324a1852c1ca885f5c79b68cd2733c5 master

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