Re: [weld-dev] tx services

I would actually think Weld could/should expose this, not CDI. As exposing TM is definitely not per spec, hence not CDI's "business". Otoh, building Tx based Seam3 components would again depend on Weld workaround; e.g. Seam Conversation. TM also exposes XAResource handling, for which TxServices don't have matching api. And like I said, there is also suspend/resume Tx API, which sometimes also comes in handy. If TxServices::getTM returns an actual TM instance, then Weld could expose it as a bean. But purely optional from the Services implementor --> null TM == no TM bean. -Ales On Apr 30, 2011, at 1:10 PM, Pete Muir wrote: > I'm not 100% sure what you mean, as TransactionServices exposes services to Weld which uses them to (a) expose UTX as a bean and (b) register for synchronizations for TX events. So just adding TM to TransactionServices won't do very much ;-), you would also need to expose TM as a bean. Weld could do this, but in general in Weld we've tried to avoid exposing built in beans (especially for standard APIs) which aren't spec defined as this could interfere with other people. So better to propose this for CDI 1.1 IMO. > > On 29 Apr 2011, at 17:26, Ales Justin wrote: > >> IMO TransactionServices should also expose TransactionManager. >> >> Since with AS7, which hides TM from JNDI, one cannot easily register XAResources: >> >> Transaction >> public boolean enlistResource(XAResource xaRes) throws RollbackException, IllegalStateException, SystemException >> >> or suspending/resuming current Tx. >> Specially XAResources handling is a must to be able to enlist if you're doing some non-trivial tx-based work. >> >> This way the integration layer would make sure the TM is exposed as a CDI service. >> >> Wdyt? >> >> -Ales >> >> >