[sqlalchemy] Re: SA 0.3.x: performance issues = proposal

2006-12-08 Thread Sébastien LELONG
As attachement, here's the patch (against rev 2132). It's local to sqlalchemy/orm/, so: cd /sqlalchemy/orm patch -p0 attribute_cache.patch About the patch itself: 1. the cache should be a WeakKeyDictionary OK, done. Performances are still OK, differences between built-in dict are

[sqlalchemy] Re: SA 0.3.x: performance issues = proposal

2006-12-06 Thread Michael Bayer
i think that is a great idea ! can you make a patch for me ? here are some things i think it should have: 1. the cache should be a WeakKeyDictionary, if possible, so that an application which creates classes on the fly doesnt run out of memory. this adds a micro-amount of overhead vs. a