[jboss-cvs] JBossAS SVN: r88366 - projects/ejb3/trunk/transactions/src/main/java/org/jboss/ejb3/tx.

[jboss-cvs-commits at lists.jboss.org]

Author: jaikiran
Date: 2009-05-07 10:55:39 -0400 (Thu, 07 May 2009)
New Revision: 88366

Modified:
   projects/ejb3/trunk/transactions/src/main/java/org/jboss/ejb3/tx/CMTTxInterceptorFactory.java
Log:
EJBTHREE-1822 Avoid repeated lookup of transaction manager. Maintain a reference to it after the first lookup

Modified: projects/ejb3/trunk/transactions/src/main/java/org/jboss/ejb3/tx/CMTTxInterceptorFactory.java
===================================================================
--- projects/ejb3/trunk/transactions/src/main/java/org/jboss/ejb3/tx/CMTTxInterceptorFactory.java	2009-05-07 14:43:34 UTC (rev 88365)
+++ projects/ejb3/trunk/transactions/src/main/java/org/jboss/ejb3/tx/CMTTxInterceptorFactory.java	2009-05-07 14:55:39 UTC (rev 88366)
@@ -43,6 +43,8 @@
 {
    @SuppressWarnings("unused")
    private static final Logger log = Logger.getLogger(CMTTxInterceptorFactory.class);
+   
+   private TransactionManager transactionManager; 
 
    protected TransactionAttributeType getTxType(Advisor advisor, Joinpoint jp)
    {
@@ -79,7 +81,7 @@
       if (type == TransactionManagementType.BEAN)
          return new NullInterceptor();
 
-      TransactionManager tm = TxUtil.getTransactionManager();
+      TransactionManager tm = this.getTransactionManager();
       
       Method method = ((MethodJoinpoint) jp).getMethod();
       int timeout = resolveTransactionTimeout(advisor, method);
@@ -122,4 +124,17 @@
          return interceptor;
       }
    }
+   
+   /**
+    * Returns the TransactionManager
+    * @return
+    */
+   private TransactionManager getTransactionManager()
+   {
+      if (this.transactionManager == null)
+      {
+         this.transactionManager = TxUtil.getTransactionManager();
+      }
+      return this.transactionManager;
+   }
 }