Author: manik.surtani(a)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)