Changes by Sebastian Bank (@xflr6):

• edited description

Michael Bayer (@zzzeek) wrote:

• The "lightweight named tuple" used when a :class:.Query returns
  rows failed to implement __slots__ correctly such that it still
  had a __dict__. This is resolved, but in the extremely
  unlikely case someone was assigning values to the returned tuples,
  that will no longer work.
  fixes LW KeyedTuples have instance dicts (ineffective __slots__) #3420

→ 64c1f2e

Changes by Michael Bayer (@zzzeek):

• changed status to closed

Michael Bayer (@zzzeek) wrote:

thanks for catching that. countdown to someone was assigning to the tuple in 5..4..

Michael Bayer (@zzzeek) wrote:

.. 3... 2.. 1.. GO !!! https://groups.google.com/forum/#!topic/sqlalchemy/syWt_EZqCbk

this user is already getting an error even without the change here because they want to set one of the actual named attributes.

Sebastian Bank (@xflr6) wrote:

Uh, you really know your users damn well!

I think Python even raises with new attributes (with and without the change).
I was curious if the change affects the benchmarks from the docs (maybe for some that would count as argument for/against __slots__).

Michael Bayer (@zzzeek) wrote:

__slots__ doesn't have a huge impact on speed, it is very useful for keeping memory under control though. The amount of memory that was being wasted throughout huge parts of SQLA on dictionaries that were hardly needed is why I put a lot of __slots__ in 1.0.

Rodrigo Menezes (@rclmenezes) wrote:

Awww mannnn. You caught me :)

I assigned some helper lambdas to the returning rows:

class MyQuery(Query):
    def __iter__(self):
        return (self.decorate(r) for r in Query.__iter__(q) )
    
    handy_lambda_1 = lambda r: stuff
    handy_lambda_2 = lambda r: more_stuff
    
    def decorate(self, row):
        row.handy_lambda_1 = handy_lambda_1
        row.handy_lambda_2 = handy_lambda_2
        return row

MySession = scoped_session(
    sessionmaker(
        extension=ZopeTransactionExtension(),
        query_cls=MyQuery
    )
)

Any chance we'll get row_cls as an option to sessionmaker? I found these helper lambdas super useful in certain scenarios.

Michael Bayer (@zzzeek) wrote:

you can use Column Bundles and assign any kind of ORM-level processing to column-loads that you'd want.

Michael Bayer (@zzzeek) wrote:

for interception of objects as they are loaded you can use the load event.

Rodrigo Menezes (@rclmenezes) wrote:

Whoa! Column bundles are great. Thanks!