[jbosscache-commits] JBoss Cache SVN: r5725 - core/trunk/src/main/java/org/jboss/cache/loader.

[jbosscache-commits at lists.jboss.org]

Author: manik.surtani at jboss.com
Date: 2008-04-28 11:02:26 -0400 (Mon, 28 Apr 2008)
New Revision: 5725

Modified:
   core/trunk/src/main/java/org/jboss/cache/loader/AdjListJDBCCacheLoader.java
Log:
TODOs

Modified: core/trunk/src/main/java/org/jboss/cache/loader/AdjListJDBCCacheLoader.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/loader/AdjListJDBCCacheLoader.java	2008-04-28 14:58:24 UTC (rev 5724)
+++ core/trunk/src/main/java/org/jboss/cache/loader/AdjListJDBCCacheLoader.java	2008-04-28 15:02:26 UTC (rev 5725)
@@ -184,27 +184,11 @@
    {
       // start a tx
       cf.prepare(tx);
-
-      try
-      {
-         put(modifications);
-
-         // commit if it's one phase only
-         if (one_phase)
-         {
-            commit(tx);
-         }
-      }
-      catch (Exception e)
-      {
-         // todo should I rollback it here or rollback is supposed to be invoke by someone from outside?
-         rollback(tx);
-         // is this ok?
-         throw e;
-      }
+      put(modifications);
+      // commit if it's one phase only
+      if (one_phase) commit(tx);
    }
 
-
    /**
     * Commits a transaction.
     *