Re: transaction_manager attribute of transaction.interfaces.IDataManager

Jim Fulton <[email protected]>
Newsgroups gmane.comp.web.zope.devel
Message-ID <[email protected]>
On Tue, Jun 8, 2010 at 6:23 AM, Laurence Rowe <[email protected]> wrote:
> On 8 June 2010 09:45, Chris Withers <[email protected]> wrote:
>> Hi All,
>>
>> What is this attribute actually used for?
>>
>> I see it present on IDataManager but I notice that zope.sqlalchemy's
>> SessionDataManager doesn't have this attribute, with no apparent ill effect.
>
>    transaction_manager = zope.interface.Attribute(
>        """The transaction manager (TM) used by this data manager.
>
>        This is a public attribute, intended for read-only use.  The value
>        is an instance of ITransactionManager, typically set by the data
>        manager's constructor.
>        """)
>
> This seems to be used only in the tests, I guess it would be useful if
> you were using multiple transaction managers in a single thread. I've
> now added it to zope.sqlalchemy's data manager.

I'm in favor of removing it from the interface.

Jim

-- 
Jim Fulton
_______________________________________________
Zope-Dev maillist  -  [email protected]
https://mail.zope.org/mailman/listinfo/zope-dev
**  No cross posts or HTML encoding!  **
(Related lists - 
 https://mail.zope.org/mailman/listinfo/zope-announce
 https://mail.zope.org/mailman/listinfo/zope )
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.