[jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-1365) Make SeamManagedEntityManagerFactory joinTransaction if available

[ http://jira.jboss.com/jira/browse/JBSEAM-1365?page=comments#action_12363491 ] Michael Youngstrom commented on JBSEAM-1365: -------------------------------------------- Well, I guess I was assuming that part of the job of managing the transaction would be to determining how the current transaction should be accessed. And currently ManagedPersistenceContext.getEntityManager() assumes the current transaciton is available through JNDI. However, what if in instead of Seam activly trying to determine if you need to join a transaction or not we could leave it up to the JPA provider and how it manages transactions by changing ManagedPersistenceContext.getEntityManager() to something like this: try { entityManager.joinTransaction(); } catch(TransactionRequiredException e) { //log something } Then it really wouldn't matter if Seam was managing the transaction or how the current transaction was being accessed. This would also fix JBSEAM-1363 for me. However, it would run the potential for a performance hit of having to handle an exception when a current transaction is not active (which would probably be more the exception than the rule). Another option might be once JBSEAM-1144 is in place the logic could look something like: If (is Seam managing transactions) { if(active) { joinTransaction } } else { try { joinTransaction } catch(TransactionRequired) { //log message } } That way you'd only get an exception thrown if Seam wasn't managing transactions and if there wasn't a current active transaction. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira