Do we really need to expose the TransactionManager's UserTransaction implementation?
Looking at the interface, it seems like a subset of TransactionManager, so couldn't we return a custom UserTransaction that just delegates to the TransactionManager? On Thu, Feb 7, 2013 at 2:06 PM, Manik Surtani <[email protected]> wrote: > Ok. Then a separate Lookup is what we'd need, I guess. Not pretty, but > oh well. > > On 7 Feb 2013, at 11:41, Galder Zamarreño <[email protected]> wrote: > > > > > On Feb 7, 2013, at 12:31 PM, Manik Surtani <[email protected]> wrote: > > > >> > >> On 7 Feb 2013, at 11:23, Galder Zamarreño <[email protected]> wrote: > >> > >>> Hi all, > >>> > >>> I'm back with a more food for thought wrt JSR-107 impl. Our > CacheManager adapter needs to implement: > >>> > >>> UserTransaction getUserTransaction(); > >>> > >>> The problem there is that there's no standard way of getting a > UserTransaction given a JTA TransactionManager. > >>> > >>> It really is down to each TransactionManager provider to give a > UserTransaction instance (whether JNDI, static…etc). > >>> > >>> So, we need a way to lookup a UserTransaction. > >>> > >>> One option is to add a getUserTransaction to TransactionManagerLookup, > but that will break existing clients. > >> > >> You mean it would break existing TML implementations? > > > > ^ Yeah, potentially yeah. > > > >> Do we know of any custom TML implementations though? > > > > Yes: > > - > https://github.com/hibernate/hibernate-orm/blob/master/hibernate-infinispan/src/main/java/org/hibernate/cache/infinispan/tm/HibernateTransactionManagerLookup.java > > - > https://github.com/jbossas/jboss-as/blob/master/clustering/infinispan/src/main/java/org/jboss/as/clustering/infinispan/TransactionManagerProvider.java > > > >> > >>> > >>> Alternatively, define a > org.infinispan.transaction.lookup.UserTransactionLookup interface which is > configurable. We'd then need to implement for existing TML classes. > >>> > >>> If anyone has any other ideas, let us know. > >>> > >>> Cheers, > >>> -- > >>> Galder Zamarreño > >>> [email protected] > >>> twitter.com/galderz > >>> > >>> Project Lead, Escalante > >>> http://escalante.io > >>> > >>> Engineer, Infinispan > >>> http://infinispan.org > >>> > >>> > >>> _______________________________________________ > >>> infinispan-dev mailing list > >>> [email protected] > >>> https://lists.jboss.org/mailman/listinfo/infinispan-dev > >> > >> -- > >> Manik Surtani > >> [email protected] > >> twitter.com/maniksurtani > >> > >> Platform Architect, JBoss Data Grid > >> http://red.ht/data-grid > >> > >> > >> _______________________________________________ > >> infinispan-dev mailing list > >> [email protected] > >> https://lists.jboss.org/mailman/listinfo/infinispan-dev > > > > > > -- > > Galder Zamarreño > > [email protected] > > twitter.com/galderz > > > > Project Lead, Escalante > > http://escalante.io > > > > Engineer, Infinispan > > http://infinispan.org > > > > > > _______________________________________________ > > infinispan-dev mailing list > > [email protected] > > https://lists.jboss.org/mailman/listinfo/infinispan-dev > > -- > Manik Surtani > [email protected] > twitter.com/maniksurtani > > Platform Architect, JBoss Data Grid > http://red.ht/data-grid > > > _______________________________________________ > infinispan-dev mailing list > [email protected] > https://lists.jboss.org/mailman/listinfo/infinispan-dev >
_______________________________________________ infinispan-dev mailing list [email protected] https://lists.jboss.org/mailman/listinfo/infinispan-dev
