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

[jbosscache-commits at lists.jboss.org]

Author: manik.surtani at jboss.com
Date: 2008-07-31 10:54:13 -0400 (Thu, 31 Jul 2008)
New Revision: 6464

Modified:
   core/trunk/src/main/java/org/jboss/cache/invocation/AbstractInvocationContext.java
Log:
TODOs

Modified: core/trunk/src/main/java/org/jboss/cache/invocation/AbstractInvocationContext.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/invocation/AbstractInvocationContext.java	2008-07-31 14:52:03 UTC (rev 6463)
+++ core/trunk/src/main/java/org/jboss/cache/invocation/AbstractInvocationContext.java	2008-07-31 14:54:13 UTC (rev 6464)
@@ -323,6 +323,7 @@
 
    public long getLockAcquisitionTimeout(long timeout)
    {
+      // TODO: this stuff really doesn't belong here.  Put it somewhere else.
       if (getOptionOverrides() != null
             && getOptionOverrides().getLockAcquisitionTimeout() >= 0)
       {
@@ -347,11 +348,13 @@
 
    public boolean isValidTransaction()
    {
+      // ought to move to the transaction context
       return transaction != null && TransactionTable.isValid(transaction);
    }
 
    public void throwIfNeeded(Throwable e) throws Throwable
    {
+      // TODO: this stuff really doesn't belong here.  Put it somewhere else.
       Option optionOverride = getOptionOverrides();
       boolean shouldRethtrow = optionOverride == null || !optionOverride.isFailSilently();
       if (!shouldRethtrow)