[storm] Pylons and Storm

[Christopher Armstrong]

On 8/13/07, Vsevolod Balashov <vsevolod at balashov.name> wrote:
>
> I think, my work is done. Result code at http://pylonshq.com/pasties/411
>
> Key features:
> - single database connection
> - cache Store objects per thread
>
> I test this code only with postgres.



I finally took a look at your code, Vsevolod. I'm really confused, because
you listed "Single database connection" as a *feature* -- why would you want
only one database connection?

If you had left out your "Database" class in the middle then the code would
be fine: Storm manages the connections itself, creating one for each Store.
The way your code is written, though, the database connection is shared
among all threads, which means you have no isolation between your threads.

-- 
Christopher Armstrong
International Man of Twistery
http://radix.twistedmatrix.com/
http://twistedmatrix.com/
http://canonical.com/
-------------- next part --------------
An HTML attachment was scrubbed...
URL: https://lists.ubuntu.com/archives/storm/attachments/20070817/2cb710be/attachment.htm