On 04/29/2010 11:21 AM, Laurence Rowe wrote:
> 2010/4/29 Chris McDonough<chr...@plope.com>:
>> On 04/29/2010 10:39 AM, Chris Withers wrote:
>>>
>>> Chris McDonough wrote:
>>>>>
>>>>> As far as I know this is unnecessary. It's not in the repoze.cluegun
>>>>> app, nor in the app I worked on at Jarn. Are there any problems when
>>>>> you remove it?
>>>>
>>>> No idea. Withers, can you confirm?
>>>
>>> With the current setup, we end up doing both a .remove() in the
>>> subscriber and then a .close() in the transaction.
>>>
>>> At worst, one of those two needs to be done.
>>>
>>> So, at worst, zope.sqlalchemy should be configurable as to whether
>>> .close or .remove is called, but hopefully (pending confirmation from
>>> Michael Bayer) just a .close should be fine and the code in the template
>>> can just go away :-)
>>
>> I need confirmation that removing it works, not that it should work.
>
> The application written by Jarn does not use it. I have other wsgi
> applications which do not use it. Zope 2 has a similar transaction
> usage, and nothing like that is ever needed for zope.sqlalchemy. It
> shouldn't be in the template.

Roger, out it goes.

- C

_______________________________________________
Repoze-dev mailing list
Repoze-dev@lists.repoze.org
http://lists.repoze.org/listinfo/repoze-dev

Reply via email to