[jbosscache-commits] JBoss Cache SVN: r5728 - in core/trunk/src: main/java/org/jboss/cache/commands and 6 other directories.

jbosscache-commits at lists.jboss.org jbosscache-commits at lists.jboss.org
Mon Apr 28 11:46:51 EDT 2008


Author: manik.surtani at jboss.com
Date: 2008-04-28 11:46:50 -0400 (Mon, 28 Apr 2008)
New Revision: 5728

Modified:
   core/trunk/src/main/java/org/jboss/cache/buddyreplication/BuddyFqnTransformer.java
   core/trunk/src/main/java/org/jboss/cache/commands/AbstractVisitor.java
   core/trunk/src/main/java/org/jboss/cache/commands/Visitor.java
   core/trunk/src/main/java/org/jboss/cache/commands/read/GetChildrenNamesCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/read/GetDataMapCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/read/GetKeyValueCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/read/GetKeysCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/read/GetNodeCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/read/GravitateDataCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/read/RemoteExistsCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/tx/CommitCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/tx/OptimisticPrepareCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/tx/PrepareCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/tx/RollbackCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/write/CreateNodeCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/write/EvictCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/write/InvalidateCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/write/MoveCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/write/PutDataMapCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/write/PutKeyValueCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/write/RemoveDataCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/write/RemoveKeyCommand.java
   core/trunk/src/main/java/org/jboss/cache/commands/write/RemoveNodeCommand.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/ActivationInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/CacheLoaderInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/CacheMgmtInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/CacheStoreInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/CallInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/DataGravitatorInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/EvictionInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/InvalidationInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/InvocationContextInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/MarshalledValueInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/NotificationInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticCreateIfNotExistsInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticLockingInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticNodeInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticReplicationInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticValidatorInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/PassivationInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/ReplicationInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/TxInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/base/PostProcessingChainedInterceptor.java
   core/trunk/src/main/java/org/jboss/cache/interceptors/base/SkipCheckChainedInterceptor.java
   core/trunk/src/test/java/org/jboss/cache/marshall/MarshalledValueTest.java
Log:
Javadocs and refactoring

Modified: core/trunk/src/main/java/org/jboss/cache/buddyreplication/BuddyFqnTransformer.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/buddyreplication/BuddyFqnTransformer.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/buddyreplication/BuddyFqnTransformer.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -62,13 +62,13 @@
    }
 
    @Override
-   public Object handleCommitCommand(InvocationContext ctx, CommitCommand commitCommand) throws Throwable
+   public Object visitCommitCommand(InvocationContext ctx, CommitCommand commitCommand) throws Throwable
    {
       return commitCommand;
    }
 
    @Override
-   public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+   public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
    {
       Fqn transformed = getBackupFqn(command.getFqn());
       return factory.buildPutDataMapCommand(null, transformed, command.getData(),
@@ -76,7 +76,7 @@
    }
 
    @Override
-   public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+   public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
    {
       Fqn transformed = getBackupFqn(command.getFqn());
       return factory.buildPutKeyValueCommand(null, transformed, command.getKey(),
@@ -84,7 +84,7 @@
    }
 
    @Override
-   public Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
+   public Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
    {
       Fqn transformed = getBackupFqn(command.getFqn());
       return factory.buildRemoveNodeCommand(command.getGlobalTransaction(), transformed, command.isEviction(),
@@ -92,7 +92,7 @@
    }
 
    @Override
-   public Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+   public Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
    {
       Fqn transformed = getBackupFqn(command.getFqn());
       return factory.buildRemoveDataCommand(command.getGlobalTransaction(), transformed, command.isCreateUndoops(),
@@ -100,70 +100,70 @@
    }
 
    @Override
-   public Object handleEvictFqnCommand(InvocationContext ctx, EvictCommand command) throws Throwable
+   public Object visitEvictFqnCommand(InvocationContext ctx, EvictCommand command) throws Throwable
    {
       Fqn fqn = getBackupFqn(command.getFqn());
       return factory.buildEvictFqnCommand(fqn);
    }
 
    @Override
-   public Object handleInvalidateCommand(InvocationContext ctx, InvalidateCommand command) throws Throwable
+   public Object visitInvalidateCommand(InvocationContext ctx, InvalidateCommand command) throws Throwable
    {
       Fqn transformed = getBackupFqn(command.getFqn());
       return factory.buildInvalidateCommand(transformed);
    }
 
    @Override
-   public Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+   public Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
    {
       Fqn transformed = getBackupFqn(command.getFqn());
       return factory.buildRemoveKeyCommand(null, transformed, command.getKey(), command.isCreateUndoOps());
    }
 
    @Override
-   public Object handleGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
+   public Object visitGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
    {
       Fqn transformed = getBackupFqn(command.getFqn());
       return factory.buildGetDataMapCommand(transformed);
    }
 
    @Override
-   public Object handleExistsNodeCommand(InvocationContext ctx, RemoteExistsCommand command) throws Throwable
+   public Object visitExistsNodeCommand(InvocationContext ctx, RemoteExistsCommand command) throws Throwable
    {
       Fqn transformed = getBackupFqn(command.getFqn());
       return factory.buildEvictFqnCommand(transformed);
    }
 
    @Override
-   public Object handleGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
+   public Object visitGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
    {
       Fqn transformed = getBackupFqn(command.getFqn());
       return factory.buildGetKeyValueCommand(transformed, command.getKey(), command.isSendNodeEvent());
    }
 
    @Override
-   public Object handleGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
+   public Object visitGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
    {
       Fqn transformed = getBackupFqn(command.getFqn());
       return factory.buildGetNodeCommand(transformed);
    }
 
    @Override
-   public Object handleGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
+   public Object visitGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
    {
       Fqn transformed = getBackupFqn(command.getFqn());
       return factory.buildGetKeysCommand(transformed);
    }
 
    @Override
-   public Object handleGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
+   public Object visitGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
    {
       Fqn transformed = getBackupFqn(command.getFqn());
       return factory.buildGetChildrenNamesCommand(transformed);
    }
 
    @Override
-   public Object handleMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
+   public Object visitMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
    {
       Fqn transformedFrom = getBackupFqn(command.getFqn());
       Fqn transformedTo = getBackupFqn(command.getTo());
@@ -171,14 +171,14 @@
    }
 
    @Override
-   public Object handleGravitateDataCommand(InvocationContext ctx, GravitateDataCommand command) throws Throwable
+   public Object visitGravitateDataCommand(InvocationContext ctx, GravitateDataCommand command) throws Throwable
    {
       Fqn transformed = getBackupFqn(command.getFqn());
       return factory.buildGravitateDataCacheCommand(transformed, command.isSearchSubtrees());
    }
 
    @Override
-   public Object handlePrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
+   public Object visitPrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
    {
       List<ReversibleCommand> toTransform = command.getModifications();
       List<ReversibleCommand> transformedCommands = transformBatch(toTransform);
@@ -186,20 +186,20 @@
    }
 
    @Override
-   public Object handleRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
+   public Object visitRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
    {
       return factory.buildRollbackCommand(null);
    }
 
    @Override
-   public Object handleOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
+   public Object visitOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
    {
       List<ReversibleCommand> transformed = transformBatch(command.getModifications());
       return factory.buildOptimisticPrepareCommand(command.getGlobalTransaction(), transformed, command.getData(), command.getLocalAddress(), command.isOnePhaseCommit());
    }
 
    @Override
-   public Object handleCreateNodeCommand(InvocationContext ctx, CreateNodeCommand command) throws Throwable
+   public Object visitCreateNodeCommand(InvocationContext ctx, CreateNodeCommand command) throws Throwable
    {
       return factory.buildCreateNodeCommand(getBackupFqn(command.getFqn()));
    }

Modified: core/trunk/src/main/java/org/jboss/cache/commands/AbstractVisitor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/AbstractVisitor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/AbstractVisitor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -25,116 +25,135 @@
 import java.util.Collection;
 
 /**
+ * An abstract implementation of a Visitor that delegates all visit calls to a default handler which can be overridden.
+ *
  * @author Mircea.Markus at jboss.com
- * @since 2.2
+ * @author Manik Surtani
+ * @since 2.2.0
  */
 public abstract class AbstractVisitor implements Visitor
 {
-   public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+   public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+   public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
+   public Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+   public Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleEvictFqnCommand(InvocationContext ctx, EvictCommand command) throws Throwable
+   public Object visitEvictFqnCommand(InvocationContext ctx, EvictCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleInvalidateCommand(InvocationContext ctx, InvalidateCommand command) throws Throwable
+   public Object visitInvalidateCommand(InvocationContext ctx, InvalidateCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+   public Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
+   public Object visitGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleExistsNodeCommand(InvocationContext ctx, RemoteExistsCommand command) throws Throwable
+   public Object visitExistsNodeCommand(InvocationContext ctx, RemoteExistsCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
+   public Object visitGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
+   public Object visitGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
+   public Object visitGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
+   public Object visitGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
+   public Object visitMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleGravitateDataCommand(InvocationContext ctx, GravitateDataCommand command) throws Throwable
+   public Object visitGravitateDataCommand(InvocationContext ctx, GravitateDataCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handlePrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
+   public Object visitPrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
+   public Object visitRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
+   public Object visitCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
+   public Object visitOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleCreateNodeCommand(InvocationContext ctx, CreateNodeCommand command) throws Throwable
+   public Object visitCreateNodeCommand(InvocationContext ctx, CreateNodeCommand command) throws Throwable
    {
       return handleDefault(ctx, command);
    }
 
-   public Object handleDefault(InvocationContext ctx, VisitableCommand command) throws Throwable
+   /**
+    * A default handler for all commands visited.  This is called for any visit method called, unless a visit command is
+    * appropriately overridden.
+    *
+    * @param ctx     invocation context
+    * @param command command to handle
+    * @return return value
+    * @throws Throwable in the case of a problem
+    */
+   protected Object handleDefault(InvocationContext ctx, VisitableCommand command) throws Throwable
    {
       return null;
    }
 
+   /**
+    * Helper method to visit a collection of VisitableCommands.
+    *
+    * @param ctx     Invocation context
+    * @param toVisit collection of commands to visit
+    * @throws Throwable in the event of problems
+    */
    public void visitCollection(InvocationContext ctx, Collection<? extends VisitableCommand> toVisit) throws Throwable
    {
       for (VisitableCommand command : toVisit)

Modified: core/trunk/src/main/java/org/jboss/cache/commands/Visitor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/Visitor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/Visitor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -23,133 +23,214 @@
 import org.jboss.cache.commands.write.RemoveNodeCommand;
 
 /**
+ * This interface is the core of JBoss Cache, where each {@link VisitableCommand} can be visited by a Visitor implementation.
+ * Visitors which are accepted by the {@link org.jboss.cache.commands.VisitableCommand} are able to modify the command
+ * based on any logic encapsulated by the visitor.
+ *
  * @author Mircea.Markus at jboss.com
- * @since 2.2
+ * @author Manik Surtani
+ * @since 2.2.0
  */
 public interface Visitor
 {
+   /**
+    * Visits a PutDataMapCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
+    */
+   Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable;
 
-   /*
-   equiv of old:
-     1 - putDataEraseMethodLocal_id
-     2 - putDataMethodLocal_id
-     3 - putDataEraseVersionedMethodLocal_id
-     4 - putDataVersionedMethodLocal_id
-   */
-
-   Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable;
-
-   /*
-     equiv of old:
-       1 - putKeyValMethodLocal_id
-       2 - putKeyValVersionedMethodLocal_id
-       3 - putForExternalReadMethodLocal_id
-       4 - putForExternalReadVersionedMethodLocal_id
+   /**
+    * Visits a PutKeyValueCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand putKeyValueCommand) throws Throwable;
+   Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable;
 
-   /*
-    equiv of old:
-       1 - removeNodeMethodLocal_id
-       2 - removeNodeVersionedMethodLocal_id
+   /**
+    * Visits a RemoveNodeCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand removeNodeCommand) throws Throwable;
+   Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable;
 
-   /*
-     equiv of old:
-       1 - removeDataMethodLocal_id
-       2 - removeDataVersionedMethodLocal_id
+   /**
+    * Visits a RemoveDataCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand removeDataCommand) throws Throwable;
+   Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable;
 
-   /*
-     equiv of old:
-       1 - evictNodeMethodLocal_id
-       2 - evictVersionedNodeMethodLocal_id
+   /**
+    * Visits a EvictCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handleEvictFqnCommand(InvocationContext ctx, EvictCommand evictFqnCommand) throws Throwable;
+   Object visitEvictFqnCommand(InvocationContext ctx, EvictCommand command) throws Throwable;
 
-   /*
-     equiv of old:
-        1 - invalidateMethodLocal_id
+   /**
+    * Visits a InvalidateCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handleInvalidateCommand(InvocationContext ctx, InvalidateCommand invalidateCommand) throws Throwable;
+   Object visitInvalidateCommand(InvocationContext ctx, InvalidateCommand command) throws Throwable;
 
-   /*
-     equiv of old:
-        1 - removeKeyMethodLocal_id
-        1 - removeKeyVersionedMethodLocal_id
+   /**
+    * Visits a RemoveKeyCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand removeKeyCommand) throws Throwable;
+   Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable;
 
-   /*
-     equiv of old:
-        1 - getDataMapMethodLocal_id
+   /**
+    * Visits a GetDataMapCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handleGetDataMapCommand(InvocationContext ctx, GetDataMapCommand getDataMapCommand) throws Throwable;
+   Object visitGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable;
 
-   /*
-     equiv of old:
-        1 - existsMethod_id
+   /**
+    * Visits a RemoteExistsCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handleExistsNodeCommand(InvocationContext ctx, RemoteExistsCommand existsNodeCommand) throws Throwable;
+   Object visitExistsNodeCommand(InvocationContext ctx, RemoteExistsCommand command) throws Throwable;
 
-   /*
-     equiv of old:
-        1 - getKeyValueMethodLocal_id
+   /**
+    * Visits a GetKeyValueCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handleGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand getKeyValueCommand) throws Throwable;
+   Object visitGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable;
 
-   /*
-     equiv of old:
-        1 - getNodeMethodLocal_id
+   /**
+    * Visits a GetNodeCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handleGetNodeCommand(InvocationContext ctx, GetNodeCommand getNodeCommand) throws Throwable;
+   Object visitGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable;
 
-   /*
-        equiv of old:
-        1 - getKeysMethodLocal_id
+   /**
+    * Visits a GetKeysCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handleGetKeysCommand(InvocationContext ctx, GetKeysCommand getKeysCommand) throws Throwable;
+   Object visitGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable;
 
-   /*
-        equiv of old:
-        1 - getChildrenNamesMethodLocal_id
+   /**
+    * Visits a GetChildrenNamesCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handleGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand getChildrenNamesCacheCommand) throws Throwable;
+   Object visitGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable;
 
-   /*
-        equiv of old:
-        1 - moveMethodLocal_id
+   /**
+    * Visits a MoveCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handleMoveCommand(InvocationContext ctx, MoveCommand moveCommand) throws Throwable;
+   Object visitMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable;
 
-   /*
-        equiv of old:
-        1 - dataGravitationMethod_id
+   /**
+    * Visits a GravitateDataCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handleGravitateDataCommand(InvocationContext ctx, GravitateDataCommand gravitateDataCommand) throws Throwable;
+   Object visitGravitateDataCommand(InvocationContext ctx, GravitateDataCommand command) throws Throwable;
 
-   /*
-      prepareMethod_id
+   /**
+    * Visits a PrepareCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handlePrepareCommand(InvocationContext ctx, PrepareCommand prepareCommand) throws Throwable;
+   Object visitPrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable;
 
-   /*
-      rollbackMethod_id
+   /**
+    * Visits a RollbackCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handleRollbackCommand(InvocationContext ctx, RollbackCommand rollbackCommand) throws Throwable;
+   Object visitRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable;
 
-   /*
-      commitMethod_id
+   /**
+    * Visits a CommitCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handleCommitCommand(InvocationContext ctx, CommitCommand commitCommand) throws Throwable;
+   Object visitCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable;
 
-   /*
-      optimisticPrepareMethod_id
+   /**
+    * Visits a OptimisticPrepareCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
     */
-   Object handleOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand optimisticPrepareCommand) throws Throwable;
+   Object visitOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable;
 
 
-   Object handleCreateNodeCommand(InvocationContext ctx, CreateNodeCommand command) throws Throwable;
+   /**
+    * Visits a CreateNodeCommand.
+    *
+    * @param ctx     invocation context
+    * @param command command to visit
+    * @return response from the visit
+    * @throws Throwable in the event of problems.
+    */
+   Object visitCreateNodeCommand(InvocationContext ctx, CreateNodeCommand command) throws Throwable;
 }

Modified: core/trunk/src/main/java/org/jboss/cache/commands/read/GetChildrenNamesCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/read/GetChildrenNamesCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/read/GetChildrenNamesCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -15,6 +15,11 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+/*
+        equiv of old:
+        1 - getChildrenNamesMethodLocal_id
+    */
+
 public class GetChildrenNamesCommand extends AbstractDataCommand
 {
    public static final int METHOD_ID = 23;
@@ -52,7 +57,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handleGetChildrenNamesCommand(ctx, this);
+      return visitor.visitGetChildrenNamesCommand(ctx, this);
    }
 
    public int getCommandId()

Modified: core/trunk/src/main/java/org/jboss/cache/commands/read/GetDataMapCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/read/GetDataMapCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/read/GetDataMapCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -13,6 +13,11 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+/*
+     equiv of old:
+        1 - getDataMapMethodLocal_id
+    */
+
 public class GetDataMapCommand extends AbstractDataCommand
 {
    public static final int METHOD_ID = 24;
@@ -35,7 +40,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handleGetDataMapCommand(ctx, this);
+      return visitor.visitGetDataMapCommand(ctx, this);
    }
 
    public int getCommandId()

Modified: core/trunk/src/main/java/org/jboss/cache/commands/read/GetKeyValueCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/read/GetKeyValueCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/read/GetKeyValueCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -15,6 +15,11 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+/*
+     equiv of old:
+        1 - getKeyValueMethodLocal_id
+    */
+
 public class GetKeyValueCommand extends AbstractDataCommand
 {
    public static final int METHOD_ID = 26;
@@ -65,7 +70,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handleGetKeyValueCommand(ctx, this);
+      return visitor.visitGetKeyValueCommand(ctx, this);
    }
 
    public Object getKey()

Modified: core/trunk/src/main/java/org/jboss/cache/commands/read/GetKeysCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/read/GetKeysCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/read/GetKeysCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -14,6 +14,11 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+/*
+        equiv of old:
+        1 - getKeysMethodLocal_id
+    */
+
 public class GetKeysCommand extends AbstractDataCommand
 {
    public static final int METHOD_ID = 25;
@@ -40,7 +45,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handleGetKeysCommand(ctx, this);
+      return visitor.visitGetKeysCommand(ctx, this);
    }
 
    public int getCommandId()

Modified: core/trunk/src/main/java/org/jboss/cache/commands/read/GetNodeCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/read/GetNodeCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/read/GetNodeCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -10,6 +10,11 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+/*
+     equiv of old:
+        1 - getNodeMethodLocal_id
+    */
+
 public class GetNodeCommand extends AbstractDataCommand
 {
    public static final int METHOD_ID = 31;
@@ -30,7 +35,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handleGetNodeCommand(ctx, this);
+      return visitor.visitGetNodeCommand(ctx, this);
    }
 
    public int getCommandId()

Modified: core/trunk/src/main/java/org/jboss/cache/commands/read/GravitateDataCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/read/GravitateDataCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/read/GravitateDataCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -25,6 +25,11 @@
  * configured to use {@link org.jboss.cache.config.Configuration#useRegionBasedMarshalling} and the {@link org.jboss.cache.Region}
  * pertaining to the Fqn passed in is activated, and has an appropriate ClassLoader.
  */
+/*
+        equiv of old:
+        1 - dataGravitationMethod_id
+    */
+
 public class GravitateDataCommand extends AbstractDataCommand
 {
    public static final int METHOD_ID = 35;
@@ -161,7 +166,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handleGravitateDataCommand(ctx, this);
+      return visitor.visitGravitateDataCommand(ctx, this);
    }
 
    public int getCommandId()

Modified: core/trunk/src/main/java/org/jboss/cache/commands/read/RemoteExistsCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/read/RemoteExistsCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/read/RemoteExistsCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -17,6 +17,11 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+/*
+     equiv of old:
+        1 - existsMethod_id
+    */
+
 public class RemoteExistsCommand extends AbstractDataCommand
 {
    public static final int METHOD_ID = 16;
@@ -38,7 +43,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handleExistsNodeCommand(ctx, this);
+      return visitor.visitExistsNodeCommand(ctx, this);
    }
 
    public int getCommandId()

Modified: core/trunk/src/main/java/org/jboss/cache/commands/tx/CommitCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/tx/CommitCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/tx/CommitCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -8,6 +8,10 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+/*
+      commitMethod_id
+    */
+
 public class CommitCommand extends AbstractTransactionCommand
 {
    public static final int METHOD_ID = 11;
@@ -23,7 +27,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handleCommitCommand(ctx, this);
+      return visitor.visitCommitCommand(ctx, this);
    }
 
    public int getCommandId()

Modified: core/trunk/src/main/java/org/jboss/cache/commands/tx/OptimisticPrepareCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/tx/OptimisticPrepareCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/tx/OptimisticPrepareCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -13,6 +13,10 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+/*
+      optimisticPrepareMethod_id
+    */
+
 public class OptimisticPrepareCommand extends PrepareCommand
 {
    public static final int METHOD_ID = 18;
@@ -31,7 +35,7 @@
    @Override
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handleOptimisticPrepareCommand(ctx, this);
+      return visitor.visitOptimisticPrepareCommand(ctx, this);
    }
 
    public Map getData()

Modified: core/trunk/src/main/java/org/jboss/cache/commands/tx/PrepareCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/tx/PrepareCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/tx/PrepareCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -14,6 +14,10 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+/*
+      prepareMethod_id
+    */
+
 public class PrepareCommand extends AbstractTransactionCommand
 {
    public static final int METHOD_ID = 10;
@@ -36,7 +40,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handlePrepareCommand(ctx, this);
+      return visitor.visitPrepareCommand(ctx, this);
    }
 
    public List<ReversibleCommand> getModifications()

Modified: core/trunk/src/main/java/org/jboss/cache/commands/tx/RollbackCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/tx/RollbackCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/tx/RollbackCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -8,6 +8,10 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+/*
+      rollbackMethod_id
+    */
+
 public class RollbackCommand extends AbstractTransactionCommand
 {
    public static final int METHOD_ID = 12;
@@ -23,7 +27,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handleRollbackCommand(ctx, this);
+      return visitor.visitRollbackCommand(ctx, this);
    }
 
    public int getCommandId()

Modified: core/trunk/src/main/java/org/jboss/cache/commands/write/CreateNodeCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/write/CreateNodeCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/write/CreateNodeCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -68,7 +68,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handleCreateNodeCommand(ctx, this);
+      return visitor.visitCreateNodeCommand(ctx, this);
    }
 
    public void rollback()

Modified: core/trunk/src/main/java/org/jboss/cache/commands/write/EvictCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/write/EvictCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/write/EvictCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -13,6 +13,12 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+/*
+     equiv of old:
+       1 - evictNodeMethodLocal_id
+       2 - evictVersionedNodeMethodLocal_id
+    */
+
 public class EvictCommand extends AbstractVersionedDataCommand
 {
    public static final int METHOD_ID = 8;
@@ -27,9 +33,10 @@
 
    /**
     * Returns false if the node still exists; i.e. was only data removed because it still has children.
+    *
     * @see org.jboss.cache.interceptors.EvictionInterceptor#handleEvictFqnCommand(org.jboss.cache.InvocationContext, EvictCommand)
-    * there is wehere this attribute is used
-   */
+    *      there is wehere this attribute is used
+    */
    public Object perform(InvocationContext ctx)
    {
       NodeSPI node = dataContainer.peek(fqn, false);
@@ -60,7 +67,8 @@
       try
       {
          return dataContainer.evict(fqn);
-      } finally
+      }
+      finally
       {
          notifier.notifyNodeEvicted(fqn, false, ctx);
       }
@@ -68,7 +76,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handleEvictFqnCommand(ctx, this);
+      return visitor.visitEvictFqnCommand(ctx, this);
    }
 
    public int getCommandId()

Modified: core/trunk/src/main/java/org/jboss/cache/commands/write/InvalidateCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/write/InvalidateCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/write/InvalidateCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -32,6 +32,11 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+/*
+     equiv of old:
+        1 - invalidateMethodLocal_id
+    */
+
 public class InvalidateCommand extends EvictCommand
 {
    public static final int METHOD_ID = 47;
@@ -155,7 +160,7 @@
    @Override
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handleInvalidateCommand(ctx, this);
+      return visitor.visitInvalidateCommand(ctx, this);
    }
 
    @Override

Modified: core/trunk/src/main/java/org/jboss/cache/commands/write/MoveCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/write/MoveCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/write/MoveCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -17,6 +17,11 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+/*
+        equiv of old:
+        1 - moveMethodLocal_id
+    */
+
 public class MoveCommand extends AbstractDataCommand implements ReversibleCommand
 {
    public static final int METHOD_ID = 36;
@@ -72,7 +77,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handleMoveCommand(ctx, this);
+      return visitor.visitMoveCommand(ctx, this);
    }
 
    public void _move(Fqn nodeToMoveFqn, Fqn newParentFqn, boolean skipNotifications, InvocationContext ctx)

Modified: core/trunk/src/main/java/org/jboss/cache/commands/write/PutDataMapCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/write/PutDataMapCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/write/PutDataMapCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -20,6 +20,14 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+/*
+   equiv of old:
+     1 - putDataEraseMethodLocal_id
+     2 - putDataMethodLocal_id
+     3 - putDataEraseVersionedMethodLocal_id
+     4 - putDataVersionedMethodLocal_id
+   */
+
 public class PutDataMapCommand extends AbstractVersionedDataCommand
 {
    public static final int METHOD_ID = 1;
@@ -86,7 +94,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handlePutDataMapCommand(ctx, this);
+      return visitor.visitPutDataMapCommand(ctx, this);
    }
 
    public Map getData()

Modified: core/trunk/src/main/java/org/jboss/cache/commands/write/PutKeyValueCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/write/PutKeyValueCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/write/PutKeyValueCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -20,6 +20,15 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+
+/*
+     equiv of old:
+       1 - putKeyValMethodLocal_id
+       2 - putKeyValVersionedMethodLocal_id
+       3 - putForExternalReadMethodLocal_id
+       4 - putForExternalReadVersionedMethodLocal_id
+    */
+
 public class PutKeyValueCommand extends AbstractVersionedDataCommand
 {
    public static final int METHOD_ID = 3;
@@ -91,7 +100,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handlePutKeyValueCommand(ctx, this);
+      return visitor.visitPutKeyValueCommand(ctx, this);
    }
 
    public boolean isPutForExternalRead()

Modified: core/trunk/src/main/java/org/jboss/cache/commands/write/RemoveDataCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/write/RemoveDataCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/write/RemoveDataCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -19,6 +19,12 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+/*
+     equiv of old:
+       1 - removeDataMethodLocal_id
+       2 - removeDataVersionedMethodLocal_id
+    */
+
 public class RemoveDataCommand extends AbstractVersionedDataCommand
 {
    public static final int METHOD_ID = 7;
@@ -120,7 +126,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handleRemoveDataCommand(ctx, this);
+      return visitor.visitRemoveDataCommand(ctx, this);
    }
 
    public boolean isEviction()

Modified: core/trunk/src/main/java/org/jboss/cache/commands/write/RemoveKeyCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/write/RemoveKeyCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/write/RemoveKeyCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -19,6 +19,12 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+/*
+     equiv of old:
+        1 - removeKeyMethodLocal_id
+        1 - removeKeyVersionedMethodLocal_id
+    */
+
 public class RemoveKeyCommand extends AbstractVersionedDataCommand
 {
    public static final int METHOD_ID = 6;
@@ -73,7 +79,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handleRemoveKeyCommand(ctx, this);
+      return visitor.visitRemoveKeyCommand(ctx, this);
    }
 
    public Object getKey()

Modified: core/trunk/src/main/java/org/jboss/cache/commands/write/RemoveNodeCommand.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/commands/write/RemoveNodeCommand.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/commands/write/RemoveNodeCommand.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -18,6 +18,12 @@
  * @author Mircea.Markus at jboss.com
  * @since 2.2
  */
+/*
+    equiv of old:
+       1 - removeNodeMethodLocal_id
+       2 - removeNodeVersionedMethodLocal_id
+    */
+
 public class RemoveNodeCommand extends AbstractVersionedDataCommand
 {
    public static final int METHOD_ID = 5;
@@ -161,7 +167,7 @@
 
    public Object acceptVisitor(InvocationContext ctx, Visitor visitor) throws Throwable
    {
-      return visitor.handleRemoveNodeCommand(ctx, this);
+      return visitor.visitRemoveNodeCommand(ctx, this);
    }
 
    public boolean isSkipSendingNodeEvents()

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/ActivationInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/ActivationInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/ActivationInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -71,9 +71,9 @@
    }
 
    @Override
-   public Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+   public Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
    {
-      Object returnValue = super.handleRemoveDataCommand(ctx, command);
+      Object returnValue = super.visitRemoveDataCommand(ctx, command);
       if (trace)
          log.trace("This is a remove data operation; removing the data from the loader, no activation processing needed.");
       loader.removeData(command.getFqn());
@@ -81,9 +81,9 @@
    }
 
    @Override
-   public Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
+   public Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
    {
-      Object returnValue = super.handleRemoveNodeCommand(ctx, command);
+      Object returnValue = super.visitRemoveNodeCommand(ctx, command);
       if (trace)
          log.trace("This is a remove operation; removing the node from the loader, no activation processing needed.");
       loader.remove(command.getFqn());
@@ -91,57 +91,57 @@
    }
 
    @Override
-   public Object handleGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
+   public Object visitGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
    {
-      Object returnValue = super.handleGetChildrenNamesCommand(ctx, command);
+      Object returnValue = super.visitGetChildrenNamesCommand(ctx, command);
       removeNodeFromCacheLoader(ctx, command.getFqn());
       return returnValue;
    }
 
    @Override
-   public Object handleGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
+   public Object visitGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
    {
-      Object returnValue = super.handleGetKeysCommand(ctx, command);
+      Object returnValue = super.visitGetKeysCommand(ctx, command);
       removeNodeFromCacheLoader(ctx, command.getFqn());
       return returnValue;
    }
 
    @Override
-   public Object handleGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
+   public Object visitGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
    {
-      Object returnValue = super.handleGetNodeCommand(ctx, command);
+      Object returnValue = super.visitGetNodeCommand(ctx, command);
       removeNodeFromCacheLoader(ctx, command.getFqn());
       return returnValue;
    }
 
    @Override
-   public Object handleGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
+   public Object visitGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
    {
-      Object returnValue = super.handleGetKeyValueCommand(ctx, command);
+      Object returnValue = super.visitGetKeyValueCommand(ctx, command);
       removeNodeFromCacheLoader(ctx, command.getFqn());
       return returnValue;
    }
 
    @Override
-   public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+   public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
    {
-      Object returnValue = super.handlePutKeyValueCommand(ctx, command);
+      Object returnValue = super.visitPutKeyValueCommand(ctx, command);
       removeNodeFromCacheLoader(ctx, command.getFqn());
       return returnValue;
    }
 
    @Override
-   public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+   public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
    {
-      Object returnValue = super.handlePutDataMapCommand(ctx, command);
+      Object returnValue = super.visitPutDataMapCommand(ctx, command);
       removeNodeFromCacheLoader(ctx, command.getFqn());
       return returnValue;
    }
 
    @Override
-   public Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+   public Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
    {
-      Object returnValue = super.handleRemoveKeyCommand(ctx, command);
+      Object returnValue = super.visitRemoveKeyCommand(ctx, command);
       removeNodeFromCacheLoader(ctx, command.getFqn());
       return returnValue;
    }
@@ -193,7 +193,7 @@
    }
 
    @Override
-   public Object handleOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
+   public Object visitOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
    {
       Object retval = invokeNextInterceptor(ctx, command);
       if (inTransaction())
@@ -209,7 +209,7 @@
    }
 
    @Override
-   public Object handlePrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
+   public Object visitPrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
    {
       Object retval = invokeNextInterceptor(ctx, command);
       if (inTransaction())
@@ -295,7 +295,7 @@
       private int txActs = 0;
 
       @Override
-      public Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand removeNodeCommand) throws Throwable
+      public Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand removeNodeCommand) throws Throwable
       {
          Modification mod = new Modification(Modification.ModificationType.REMOVE_NODE, removeNodeCommand.getFqn());
          cacheLoaderModifications.add(mod);
@@ -303,7 +303,7 @@
       }
 
       @Override
-      public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+      public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
       {
          Fqn fqn = command.getFqn();
          handlePutCommand(ctx, fqn);
@@ -344,9 +344,9 @@
 
 
       @Override
-      public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand putKeyValueCommand) throws Throwable
+      public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand putKeyValueCommand) throws Throwable
       {
-         return super.handlePutKeyValueCommand(ctx, putKeyValueCommand);
+         return super.visitPutKeyValueCommand(ctx, putKeyValueCommand);
       }
 
       private boolean loaderNoChildren(Fqn fqn) throws Exception

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/CacheLoaderInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/CacheLoaderInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/CacheLoaderInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -90,7 +90,7 @@
    }
 
    @Override
-   public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+   public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
    {
       if (command.getFqn() != null)
       {
@@ -100,7 +100,7 @@
    }
 
    @Override
-   public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+   public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
    {
       if (command.getFqn() != null)
       {
@@ -110,7 +110,7 @@
    }
 
    @Override
-   public Object handleMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
+   public Object visitMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
    {
       if (command.getFqn() != null)
       {
@@ -124,7 +124,7 @@
    }
 
    @Override
-   public Object handleGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
+   public Object visitGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
    {
       if (command.getFqn() != null)
       {
@@ -135,7 +135,7 @@
 
 
    @Override
-   public Object handleGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
+   public Object visitGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
    {
       if (command.getFqn() != null)
       {
@@ -145,7 +145,7 @@
    }
 
    @Override
-   public Object handleGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
+   public Object visitGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
    {
       Fqn fqn = command.getFqn();
       if (fqn != null)
@@ -159,7 +159,7 @@
 
 
    @Override
-   public Object handleGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
+   public Object visitGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
    {
       if (command.getFqn() != null)
       {
@@ -169,7 +169,7 @@
    }
 
    @Override
-   public Object handleGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
+   public Object visitGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
    {
       if (command.getFqn() != null)
       {
@@ -179,7 +179,7 @@
    }
 
    @Override
-   public Object handleRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
+   public Object visitRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
    {
       // clean up nodesCreated map
       if (trace) log.trace("Removing temporarily created nodes from treecache");
@@ -206,7 +206,7 @@
    }
 
    @Override
-   public Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
+   public Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
    {
       if (config.isNodeLockingOptimistic() && command.getFqn() != null)
       {
@@ -216,7 +216,7 @@
    }
 
    @Override
-   public Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+   public Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
    {
       if (command.getFqn() != null && !useCacheStore)
       {
@@ -226,7 +226,7 @@
    }
 
    @Override
-   public Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+   public Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
    {
       Fqn fqn = command.getFqn();
       if (fqn != null && !useCacheStore)

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/CacheMgmtInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/CacheMgmtInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/CacheMgmtInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -60,7 +60,7 @@
    }
 
    @Override
-   public Object handleEvictFqnCommand(InvocationContext ctx, EvictCommand command) throws Throwable
+   public Object visitEvictFqnCommand(InvocationContext ctx, EvictCommand command) throws Throwable
    {
       Object returnValue = invokeNextInterceptor(ctx, command);
       m_evictions++;
@@ -68,7 +68,7 @@
    }
 
    @Override
-   public Object handleGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
+   public Object visitGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
    {
       long t1 = System.currentTimeMillis();
       Object retval = invokeNextInterceptor(ctx, command);
@@ -87,7 +87,7 @@
    }
 
    @Override
-   public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+   public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
    {
       Map data = command.getData();
       long t1 = System.currentTimeMillis();
@@ -104,7 +104,7 @@
 
 
    @Override
-   public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+   public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
    {
       long t1 = System.currentTimeMillis();
       Object retval = invokeNextInterceptor(ctx, command);

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/CacheStoreInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/CacheStoreInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/CacheStoreInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -406,7 +406,7 @@
       }
 
       @Override
-      public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+      public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
       {
          if (generateStatistics) putCount++;
          if (command.isEraseContents())
@@ -422,7 +422,7 @@
       }
 
       @Override
-      public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+      public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
       {
          if (generateStatistics) putCount++;
          modifications.add(new Modification(Modification.ModificationType.PUT_KEY_VALUE, command.getFqn(),
@@ -432,7 +432,7 @@
       }
 
       @Override
-      public Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+      public Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
       {
          modifications.add(new Modification(Modification.ModificationType.REMOVE_KEY_VALUE, command.getFqn(), command.getKey()));
          affectedFqns.add(command.getFqn());
@@ -440,7 +440,7 @@
       }
 
       @Override
-      public Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+      public Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
       {
          modifications.add(new Modification(Modification.ModificationType.REMOVE_DATA, command.getFqn()));
          affectedFqns.add(command.getFqn());
@@ -448,7 +448,7 @@
       }
 
       @Override
-      public Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
+      public Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
       {
          modifications.add(new Modification(Modification.ModificationType.REMOVE_NODE, command.getFqn()));
          affectedFqns.add(command.getFqn());
@@ -456,7 +456,7 @@
       }
 
       @Override
-      public Object handleMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
+      public Object visitMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
       {
          Fqn moveFrom = command.getFqn();
          affectedFqns.add(command.getFqn());

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/CallInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/CallInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/CallInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -60,28 +60,28 @@
    }
 
    @Override
-   public Object handlePrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
+   public Object visitPrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
    {
       if (trace) log.trace("Suppressing invocation of method handlePrepareCommand.");
       return null;
    }
 
    @Override
-   public Object handleOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
+   public Object visitOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
    {
       if (trace) log.trace("Suppressing invocation of method handleOptimisticPrepareCommand.");
       return null;
    }
 
    @Override
-   public Object handleCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
+   public Object visitCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
    {
       if (trace) log.trace("Suppressing invocation of method handleCommitCommand.");
       return null;
    }
 
    @Override
-   public Object handleRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
+   public Object visitRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
    {
       if (trace) log.trace("Suppressing invocation of method handleRollbackCommand.");
       return null;
@@ -115,37 +115,37 @@
    }
 
    @Override
-   public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+   public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
    {
       return handleAlterCacheMethod(ctx, command);
    }
 
    @Override
-   public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+   public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
    {
       return handleAlterCacheMethod(ctx, command);
    }
 
    @Override
-   public Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
+   public Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
    {
       return handleAlterCacheMethod(ctx, command);
    }
 
    @Override
-   public Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+   public Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
    {
       return handleAlterCacheMethod(ctx, command);
    }
 
    @Override
-   public Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+   public Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
    {
       return handleAlterCacheMethod(ctx, command);
    }
 
    @Override
-   public Object handleMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
+   public Object visitMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
    {
       return handleAlterCacheMethod(ctx, command);
    }

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/DataGravitatorInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/DataGravitatorInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/DataGravitatorInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -95,43 +95,43 @@
    }
 
    @Override
-   public Object handleGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
+   public Object visitGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
    {
       return handleGetMethod(ctx, command);
    }
 
    @Override
-   public Object handleGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
+   public Object visitGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
    {
       return handleGetMethod(ctx, command);
    }
 
    @Override
-   public Object handleExistsNodeCommand(InvocationContext ctx, RemoteExistsCommand command) throws Throwable
+   public Object visitExistsNodeCommand(InvocationContext ctx, RemoteExistsCommand command) throws Throwable
    {
       return handleGetMethod(ctx, command);
    }
 
    @Override
-   public Object handleGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
+   public Object visitGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
    {
       return handleGetMethod(ctx, command);
    }
 
    @Override
-   public Object handleGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
+   public Object visitGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
    {
       return handleGetMethod(ctx, command);
    }
 
    @Override
-   public Object handleGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
+   public Object visitGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
    {
       return handleGetMethod(ctx, command);
    }
 
    @Override
-   public Object handleRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
+   public Object visitRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
    {
       try
       {
@@ -144,7 +144,7 @@
    }
 
    @Override
-   public Object handleCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
+   public Object visitCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
    {
       GlobalTransaction gtx = ctx.getGlobalTransaction();
       try

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/EvictionInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/EvictionInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/EvictionInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -60,7 +60,7 @@
    }
 
    @Override
-   public Object handleEvictFqnCommand(InvocationContext ctx, EvictCommand command) throws Throwable
+   public Object visitEvictFqnCommand(InvocationContext ctx, EvictCommand command) throws Throwable
    {
       Fqn fqn = command.getFqn();
       Object retVal = invokeNextInterceptor(ctx, command);
@@ -80,7 +80,7 @@
    }
 
    @Override
-   public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+   public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
    {
       Object retVal = invokeNextInterceptor(ctx, command);
       Region r;
@@ -92,7 +92,7 @@
    }
 
    @Override
-   public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+   public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
    {
       Object retVal = invokeNextInterceptor(ctx, command);
       Fqn fqn = command.getFqn();
@@ -122,7 +122,7 @@
    }
 
    @Override
-   public Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+   public Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
    {
       Object retVal = invokeNextInterceptor(ctx, command);
       if (retVal == null)
@@ -146,7 +146,7 @@
    }
 
    @Override
-   public Object handleGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
+   public Object visitGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
    {
       Object retVal = invokeNextInterceptor(ctx, command);
       return handleGetNodeOrDataCommands(ctx, retVal, command.getFqn());
@@ -173,14 +173,14 @@
    }
 
    @Override
-   public Object handleGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
+   public Object visitGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
    {
       Object retVal = invokeNextInterceptor(ctx, command);
       return handleGetNodeOrDataCommands(ctx, retVal, command.getFqn());
    }
 
    @Override
-   public Object handleGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
+   public Object visitGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
    {
       Object retVal = invokeNextInterceptor(ctx, command);
       Fqn fqn = command.getFqn();
@@ -200,7 +200,7 @@
    }
 
    @Override
-   public Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
+   public Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
    {
       Object retVal = invokeNextInterceptor(ctx, command);
       Region r;
@@ -212,7 +212,7 @@
    }
 
    @Override
-   public Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+   public Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
    {
       Object retVal = invokeNextInterceptor(ctx, command);
       Region r;

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/InvalidationInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/InvalidationInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/InvalidationInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -100,44 +100,44 @@
    }
 
    @Override
-   public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+   public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
    {
       return handleWriteMethod(ctx, command.getFqn(), null, command);
    }
 
    @Override
-   public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+   public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
    {
       if (command.isPutForExternalRead()) return invokeNextInterceptor(ctx, command);
       return handleWriteMethod(ctx, command.getFqn(), null, command);
    }
 
    @Override
-   public Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
+   public Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
    {
       return handleWriteMethod(ctx, command.getFqn(), null, command);
    }
 
    @Override
-   public Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+   public Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
    {
       return handleWriteMethod(ctx, command.getFqn(), null, command);
    }
 
    @Override
-   public Object handleMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
+   public Object visitMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
    {
       return handleWriteMethod(ctx, command.getTo(), command.getFqn(), command);
    }
 
    @Override
-   public Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+   public Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
    {
       return handleWriteMethod(ctx, command.getFqn(), null, command);
    }
 
    @Override
-   public Object handlePrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
+   public Object visitPrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
    {
       Object retval = invokeNextInterceptor(ctx, command);
       Transaction tx = ctx.getTransaction();
@@ -162,7 +162,7 @@
    }
 
    @Override
-   public Object handleOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
+   public Object visitOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
    {
       Object retval = invokeNextInterceptor(ctx, command);
       Transaction tx = ctx.getTransaction();
@@ -182,7 +182,7 @@
    }
 
    @Override
-   public Object handleCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
+   public Object visitCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
    {
       Object retval = invokeNextInterceptor(ctx, command);
       Transaction tx = ctx.getTransaction();
@@ -197,7 +197,7 @@
    }
 
    @Override
-   public Object handleRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
+   public Object visitRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
    {
       Object retval = invokeNextInterceptor(ctx, command);
       Transaction tx = ctx.getTransaction();
@@ -290,7 +290,7 @@
       }
 
       @Override
-      public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+      public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
       {
          if (command.isPutForExternalRead())
          {
@@ -302,31 +302,31 @@
       }
 
       @Override
-      public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+      public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
       {
          return addFqn(command.getFqn());
       }
 
       @Override
-      public Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
+      public Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
       {
          return addFqn(command.getFqn());
       }
 
       @Override
-      public Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+      public Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
       {
          return addFqn(command.getFqn());
       }
 
       @Override
-      public Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+      public Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
       {
          return addFqn(command.getFqn());
       }
 
       @Override
-      public Object handleMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
+      public Object visitMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
       {
          if (containsPutForExternalRead) return null;
          result.add(command.getFqn());

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/InvocationContextInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/InvocationContextInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/InvocationContextInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -44,55 +44,55 @@
    }
 
    @Override
-   public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+   public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
    {
       return handleAll(ctx, command, command.getGlobalTransaction());
    }
 
    @Override
-   public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+   public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
    {
       return handleAll(ctx, command, command.getGlobalTransaction());
    }
 
    @Override
-   public Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
+   public Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
    {
       return handleAll(ctx, command, command.getGlobalTransaction());
    }
 
    @Override
-   public Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+   public Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
    {
       return handleAll(ctx, command, command.getGlobalTransaction());
    }
 
    @Override
-   public Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+   public Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
    {
       return handleAll(ctx, command, command.getGlobalTransaction());
    }
 
    @Override
-   public Object handlePrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
+   public Object visitPrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
    {
       return handleAll(ctx, command, command.getGlobalTransaction());
    }
 
    @Override
-   public Object handleRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
+   public Object visitRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
    {
       return handleAll(ctx, command, command.getGlobalTransaction());
    }
 
    @Override
-   public Object handleCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
+   public Object visitCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
    {
       return handleAll(ctx, command, command.getGlobalTransaction());
    }
 
    @Override
-   public Object handleOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
+   public Object visitOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
    {
       return handleAll(ctx, command, command.getGlobalTransaction());
    }

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/MarshalledValueInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/MarshalledValueInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/MarshalledValueInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -38,7 +38,7 @@
 {
 
    @Override
-   public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+   public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
    {
       if (command.getDataVersion() != null)
       {
@@ -51,7 +51,7 @@
    }
 
    @Override
-   public Object handleGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
+   public Object visitGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
    {
       Object retVal = invokeNextInterceptor(ctx, command);
       if (retVal instanceof Map)
@@ -64,7 +64,7 @@
    }
 
    @Override
-   public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+   public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
    {
       Set<MarshalledValue> marshalledValues = new HashSet<MarshalledValue>();
       if (!MarshalledValueHelper.isTypeExcluded(command.getKey().getClass()))
@@ -82,21 +82,21 @@
    }
 
    @Override
-   public Object handleGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
+   public Object visitGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
    {
       Object retVal = invokeNextInterceptor(ctx, command);
       return processRetVal(retVal);
    }
 
    @Override
-   public Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+   public Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
    {
       Object retVal = invokeNextInterceptor(ctx, command);
       return processRetVal(retVal);
    }
 
    @Override
-   public Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+   public Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
    {
       Set<MarshalledValue> marshalledValues = new HashSet<MarshalledValue>();
       if (!MarshalledValueHelper.isTypeExcluded(command.getKey().getClass()))
@@ -109,21 +109,21 @@
    }
 
    @Override
-   public Object handleGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
+   public Object visitGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
    {
       Object retVal = invokeNextInterceptor(ctx, command);
       return processRetVal(retVal);
    }
 
    @Override
-   public Object handleGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
+   public Object visitGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
    {
       Object retVal = invokeNextInterceptor(ctx, command);
       return processRetVal(retVal);
    }
 
    @Override
-   public Object handleGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
+   public Object visitGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
    {
       Set<MarshalledValue> marshalledValues = new HashSet<MarshalledValue>();
       if (!MarshalledValueHelper.isTypeExcluded(command.getKey().getClass()))

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/NotificationInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/NotificationInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/NotificationInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -26,7 +26,7 @@
    }
 
    @Override
-   public Object handleCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
+   public Object visitCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
    {
       Object retval = invokeNextInterceptor(ctx, command);
       notifier.notifyTransactionCompleted(ctx.getTransaction(), true, ctx);
@@ -34,7 +34,7 @@
    }
 
    @Override
-   public Object handleRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
+   public Object visitRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
    {
       Object retval = invokeNextInterceptor(ctx, command);
       notifier.notifyTransactionCompleted(ctx.getTransaction(), false, ctx);

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticCreateIfNotExistsInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticCreateIfNotExistsInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticCreateIfNotExistsInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -60,21 +60,21 @@
    }
 
    @Override
-   public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+   public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
    {
       createNode(ctx, command.getFqn(), false);
       return invokeNextInterceptor(ctx, command);
    }
 
    @Override
-   public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+   public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
    {
       createNode(ctx, command.getFqn(), false);
       return invokeNextInterceptor(ctx, command);
    }
 
    @Override
-   public Object handleMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
+   public Object visitMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
    {
       List<Fqn> fqns = new ArrayList<Fqn>();
       fqns.add(command.getTo());

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticLockingInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticLockingInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticLockingInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -53,7 +53,7 @@
    }
 
    @Override
-   public Object handleOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
+   public Object visitOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
    {
       //try and acquire the locks - before passing on
       GlobalTransaction gtx = getGlobalTransaction(ctx);
@@ -109,13 +109,13 @@
    }
 
    @Override
-   public Object handleCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
+   public Object visitCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
    {
       return transactionFinalized(ctx, command);
    }
 
    @Override
-   public Object handleRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
+   public Object visitRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
    {
       return transactionFinalized(ctx, command);
    }

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticNodeInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticNodeInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticNodeInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -79,7 +79,7 @@
    }
 
    @Override
-   public Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
+   public Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
    {
       GlobalTransaction gtx = getGlobalTransaction(ctx);
       TransactionWorkspace workspace = getTransactionWorkspace(gtx);
@@ -94,7 +94,7 @@
    }
 
    @Override
-   public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+   public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
    {
       GlobalTransaction gtx = getGlobalTransaction(ctx);
       TransactionWorkspace workspace = getTransactionWorkspace(gtx);
@@ -117,7 +117,7 @@
    }
 
    @Override
-   public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+   public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
    {
       GlobalTransaction gtx = getGlobalTransaction(ctx);
       TransactionWorkspace workspace = getTransactionWorkspace(gtx);
@@ -141,7 +141,7 @@
 
 
    @Override
-   public Object handleMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
+   public Object visitMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
    {
       GlobalTransaction gtx = getGlobalTransaction(ctx);
       TransactionWorkspace workspace = getTransactionWorkspace(gtx);
@@ -160,7 +160,7 @@
    }
 
    @Override
-   public Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+   public Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
    {
       GlobalTransaction gtx = getGlobalTransaction(ctx);
       TransactionWorkspace workspace = getTransactionWorkspace(gtx);
@@ -176,7 +176,7 @@
    }
 
    @Override
-   public Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+   public Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
    {
       GlobalTransaction gtx = getGlobalTransaction(ctx);
       TransactionWorkspace workspace = getTransactionWorkspace(gtx);
@@ -191,7 +191,7 @@
    }
 
    @Override
-   public Object handleGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
+   public Object visitGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
    {
       TransactionWorkspace workspace = getTransactionWorkspace(getGlobalTransaction(ctx));
       Object result;
@@ -215,7 +215,7 @@
    }
 
    @Override
-   public Object handleGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
+   public Object visitGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
    {
       TransactionWorkspace workspace = getTransactionWorkspace(getGlobalTransaction(ctx));
       Object result;
@@ -238,7 +238,7 @@
    }
 
    @Override
-   public Object handleGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
+   public Object visitGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
    {
       TransactionWorkspace workspace = getTransactionWorkspace(getGlobalTransaction(ctx));
       Object result;
@@ -260,7 +260,7 @@
    }
 
    @Override
-   public Object handleGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
+   public Object visitGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
    {
       TransactionWorkspace workspace = getTransactionWorkspace(getGlobalTransaction(ctx));
       Object result;
@@ -282,7 +282,7 @@
    }
 
    @Override
-   public Object handleGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
+   public Object visitGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
    {
       TransactionWorkspace workspace = getTransactionWorkspace(getGlobalTransaction(ctx));
       Object result;

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticReplicationInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticReplicationInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticReplicationInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -88,7 +88,7 @@
    }
 
    @Override
-   public Object handleOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
+   public Object visitOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
    {
       // pass up the chain.
       Object retval = invokeNextInterceptor(ctx, command);
@@ -106,7 +106,7 @@
    }
 
    @Override
-   public Object handleCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
+   public Object visitCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
    {
       if (isLocalOptionOverrides(ctx))
       {
@@ -138,7 +138,7 @@
    }
 
    @Override
-   public Object handleRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
+   public Object visitRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
    {
       if (isLocalOptionOverrides(ctx))
       {
@@ -170,7 +170,7 @@
    }
 
    @Override
-   public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+   public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
    {
       if (isLocalOptionOverrides(ctx))
       {
@@ -291,13 +291,13 @@
       }
 
       @Override
-      public Object handleGravitateDataCommand(InvocationContext ctx, GravitateDataCommand command) throws Throwable
+      public Object visitGravitateDataCommand(InvocationContext ctx, GravitateDataCommand command) throws Throwable
       {
          return command;
       }
 
       @Override
-      public Object handleEvictFqnCommand(InvocationContext ctx, EvictCommand command) throws Throwable
+      public Object visitEvictFqnCommand(InvocationContext ctx, EvictCommand command) throws Throwable
       {
          EvictCommand clone = commandsFactory.buildEvictFqnCommand(command.getFqn());
          return setDataVersion(clone, command.getFqn());
@@ -311,21 +311,21 @@
       }
 
       @Override
-      public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+      public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
       {
          PutDataMapCommand clone = commandsFactory.buildPutDataMapCommand(null, command.getFqn(), command.getData(), command.isCreateUndoOps(), command.isEraseContents());
          return setDataVersion(clone, clone.getFqn());
       }
 
       @Override
-      public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+      public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
       {
          PutKeyValueCommand clone = commandsFactory.buildPutKeyValueCommand(null, command.getFqn(), command.getKey(), command.getValue(), command.isCreateUndoOps(), command.isPutForExternalRead());
          return setDataVersion(clone, command.getFqn());
       }
 
       @Override
-      public Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
+      public Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
       {
          RemoveNodeCommand clone = commandsFactory.buildRemoveNodeCommand(command.getGlobalTransaction(), command.getFqn(), command.isEviction(),
                command.isSkipSendingNodeEvents(), command.isCreateUndoOps());
@@ -333,14 +333,14 @@
       }
 
       @Override
-      public Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+      public Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
       {
          RemoveKeyCommand clone = commandsFactory.buildRemoveKeyCommand(null, command.getFqn(), command.getKey(), command.isCreateUndoOps());
          return setDataVersion(clone, command.getFqn());
       }
 
       @Override
-      public Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+      public Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
       {
          RemoveDataCommand clone = commandsFactory.buildRemoveDataCommand(command.getGlobalTransaction(), command.getFqn(), command.isCreateUndoops(),
                command.isSendNodeEvent(), command.isEviction());
@@ -348,13 +348,13 @@
       }
 
       @Override
-      public Object handleCreateNodeCommand(InvocationContext ctx, CreateNodeCommand command) throws Throwable
+      public Object visitCreateNodeCommand(InvocationContext ctx, CreateNodeCommand command) throws Throwable
       {
          return command;
       }
 
       @Override
-      public Object handleMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
+      public Object visitMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
       {
          return command;
       }

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticValidatorInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticValidatorInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/OptimisticValidatorInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -62,7 +62,7 @@
    }
 
    @Override
-   public Object handleOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
+   public Object visitOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
    {
       TransactionWorkspace workspace = getTransactionWorkspace(getGlobalTransaction(ctx));
 
@@ -136,7 +136,7 @@
    }
 
    @Override
-   public Object handleCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
+   public Object visitCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
    {
       GlobalTransaction gtx = getGlobalTransaction(ctx);
       TransactionWorkspace workspace;
@@ -253,7 +253,7 @@
    }
 
    @Override
-   public Object handleRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
+   public Object visitRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
    {
       TransactionWorkspace workspace;
       workspace = getTransactionWorkspace(getGlobalTransaction(ctx));

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/PassivationInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/PassivationInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/PassivationInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -49,7 +49,7 @@
     * store using the CacheLoader.
     */
    @Override
-   public Object handleEvictFqnCommand(InvocationContext ctx, EvictCommand evictFqnCommand) throws Throwable
+   public Object visitEvictFqnCommand(InvocationContext ctx, EvictCommand evictFqnCommand) throws Throwable
    {
       Fqn fqn = evictFqnCommand.getFqn();
       try

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/ReplicationInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/ReplicationInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/ReplicationInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -47,7 +47,7 @@
    }
 
    @Override
-   public Object handleCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
+   public Object visitCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
    {
       if (!skipReplicationOfTransactionMethod(ctx))
          replicateCall(ctx, command, config.isSyncCommitPhase(), ctx.getOptionOverrides(), true);
@@ -55,7 +55,7 @@
    }
 
    @Override
-   public Object handlePrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
+   public Object visitPrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
    {
       Object retVal = invokeNextInterceptor(ctx, command);
       if (!skipReplicationOfTransactionMethod(ctx)) runPreparePhase(command, command.getGlobalTransaction(), ctx);
@@ -63,7 +63,7 @@
    }
 
    @Override
-   public Object handleRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
+   public Object visitRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
    {
       if (!skipReplicationOfTransactionMethod(ctx) && !ctx.isLocalRollbackOnly())
       {
@@ -73,7 +73,7 @@
    }
 
    @Override
-   public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+   public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
    {
       if (skipReplication(ctx)) return invokeNextInterceptor(ctx, command);
       if (isTransactionalAndLocal(ctx))
@@ -92,31 +92,31 @@
    }
 
    @Override
-   public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+   public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
    {
       return handleCrudMethod(ctx, command, false);
    }
 
    @Override
-   public Object handleMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
+   public Object visitMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
    {
       return handleCrudMethod(ctx, command, false);
    }
 
    @Override
-   public Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
+   public Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
    {
       return handleCrudMethod(ctx, command, false);
    }
 
    @Override
-   public Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+   public Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
    {
       return handleCrudMethod(ctx, command, false);
    }
 
    @Override
-   public Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+   public Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
    {
       return handleCrudMethod(ctx, command, false);
    }

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/TxInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/TxInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/TxInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -99,13 +99,13 @@
 
    @Override
    @SuppressWarnings("unchecked")
-   public Object handleOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
+   public Object visitOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
    {
-      return handlePrepareCommand(ctx, command);
+      return visitPrepareCommand(ctx, command);
    }
 
    @Override
-   public Object handlePrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
+   public Object visitPrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
    {
       Object result = null;
       boolean scrubTxsOnExit = false;
@@ -147,7 +147,7 @@
 
    @Override
    @SuppressWarnings("unchecked")
-   public Object handleCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
+   public Object visitCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
    {
       if (!ctx.getGlobalTransaction().isRemote())
       {
@@ -202,7 +202,7 @@
    }
 
    @Override
-   public Object handleRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
+   public Object visitRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
    {
       if (!ctx.getGlobalTransaction().isRemote())
       {
@@ -398,7 +398,7 @@
 
 
    @Override
-   public Object handleInvalidateCommand(InvocationContext ctx, InvalidateCommand command) throws Throwable
+   public Object visitInvalidateCommand(InvocationContext ctx, InvalidateCommand command) throws Throwable
    {
       return invokeNextInterceptor(ctx, command);
    }
@@ -650,31 +650,31 @@
       }
 
       @Override
-      public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+      public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
       {
          return handleDataVersionCommand(ctx, command);
       }
 
       @Override
-      public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+      public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
       {
          return handleDataVersionCommand(ctx, command);
       }
 
       @Override
-      public Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
+      public Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
       {
          return handleDataVersionCommand(ctx, command);
       }
 
       @Override
-      public Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+      public Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
       {
          return handleDataVersionCommand(ctx, command);
       }
 
       @Override
-      public Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+      public Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
       {
          return handleDataVersionCommand(ctx, command);
       }
@@ -1040,63 +1040,63 @@
       m.acceptVisitor(null, new AbstractVisitor()
       {
          @Override
-         public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+         public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
          {
             command.setGlobalTransaction(gtx);
             return null;
          }
 
          @Override
-         public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+         public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
          {
             command.setGlobalTransaction(gtx);
             return null;
          }
 
          @Override
-         public Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+         public Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
          {
             command.setGlobalTransaction(gtx);
             return null;
          }
 
          @Override
-         public Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
+         public Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
          {
             command.setGlobalTransaction(gtx);
             return null;
          }
 
          @Override
-         public Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+         public Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
          {
             command.setGlobalTransaction(gtx);
             return null;
          }
 
          @Override
-         public Object handleCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
+         public Object visitCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
          {
             command.setGlobalTransaction(gtx);
             return null;
          }
 
          @Override
-         public Object handlePrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
+         public Object visitPrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
          {
             command.setGlobalTransaction(gtx);
             return null;
          }
 
          @Override
-         public Object handleRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
+         public Object visitRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
          {
             command.setGlobalTransaction(gtx);
             return null;
          }
 
          @Override
-         public Object handleOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
+         public Object visitOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
          {
             command.setGlobalTransaction(gtx);
             return null;

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/base/PostProcessingChainedInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/base/PostProcessingChainedInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/base/PostProcessingChainedInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -32,7 +32,7 @@
 public abstract class PostProcessingChainedInterceptor extends CommandInterceptor
 {
    @Override
-   public final Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+   public final Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
    {
       try
       {
@@ -50,7 +50,7 @@
    }
 
    @Override
-   public final Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+   public final Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
    {
       try
       {
@@ -68,7 +68,7 @@
    }
 
    @Override
-   public final Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
+   public final Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
    {
       try
       {
@@ -86,7 +86,7 @@
    }
 
    @Override
-   public final Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+   public final Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
    {
       try
       {
@@ -104,7 +104,7 @@
    }
 
    @Override
-   public final Object handleEvictFqnCommand(InvocationContext ctx, EvictCommand command) throws Throwable
+   public final Object visitEvictFqnCommand(InvocationContext ctx, EvictCommand command) throws Throwable
    {
       try
       {
@@ -122,7 +122,7 @@
    }
 
    @Override
-   public final Object handleInvalidateCommand(InvocationContext ctx, InvalidateCommand command) throws Throwable
+   public final Object visitInvalidateCommand(InvocationContext ctx, InvalidateCommand command) throws Throwable
    {
       try
       {
@@ -140,7 +140,7 @@
    }
 
    @Override
-   public final Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+   public final Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
    {
       try
       {
@@ -158,7 +158,7 @@
    }
 
    @Override
-   public final Object handleGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
+   public final Object visitGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
    {
       try
       {
@@ -176,7 +176,7 @@
    }
 
    @Override
-   public final Object handleExistsNodeCommand(InvocationContext ctx, RemoteExistsCommand command) throws Throwable
+   public final Object visitExistsNodeCommand(InvocationContext ctx, RemoteExistsCommand command) throws Throwable
    {
       try
       {
@@ -194,7 +194,7 @@
    }
 
    @Override
-   public final Object handleGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
+   public final Object visitGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
    {
       try
       {
@@ -212,7 +212,7 @@
    }
 
    @Override
-   public final Object handleGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
+   public final Object visitGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
    {
       try
       {
@@ -230,7 +230,7 @@
    }
 
    @Override
-   public final Object handleGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
+   public final Object visitGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
    {
       try
       {
@@ -248,7 +248,7 @@
    }
 
    @Override
-   public final Object handleGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
+   public final Object visitGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
    {
       try
       {
@@ -266,7 +266,7 @@
    }
 
    @Override
-   public final Object handleMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
+   public final Object visitMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
    {
       try
       {
@@ -284,7 +284,7 @@
    }
 
    @Override
-   public final Object handleGravitateDataCommand(InvocationContext ctx, GravitateDataCommand command) throws Throwable
+   public final Object visitGravitateDataCommand(InvocationContext ctx, GravitateDataCommand command) throws Throwable
    {
       try
       {
@@ -302,7 +302,7 @@
    }
 
    @Override
-   public final Object handlePrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
+   public final Object visitPrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
    {
       try
       {
@@ -320,7 +320,7 @@
    }
 
    @Override
-   public final Object handleRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
+   public final Object visitRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
    {
       try
       {
@@ -338,7 +338,7 @@
    }
 
    @Override
-   public final Object handleCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
+   public final Object visitCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
    {
       try
       {
@@ -356,7 +356,7 @@
    }
 
    @Override
-   public final Object handleOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
+   public final Object visitOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
    {
       try
       {

Modified: core/trunk/src/main/java/org/jboss/cache/interceptors/base/SkipCheckChainedInterceptor.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/interceptors/base/SkipCheckChainedInterceptor.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/main/java/org/jboss/cache/interceptors/base/SkipCheckChainedInterceptor.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -33,7 +33,7 @@
 public abstract class SkipCheckChainedInterceptor extends CommandInterceptor
 {
    @Override
-   public final Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+   public final Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -48,7 +48,7 @@
    }
 
    @Override
-   public final Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+   public final Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -63,7 +63,7 @@
    }
 
    @Override
-   public final Object handleRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
+   public final Object visitRemoveNodeCommand(InvocationContext ctx, RemoveNodeCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -78,7 +78,7 @@
    }
 
    @Override
-   public final Object handleRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
+   public final Object visitRemoveDataCommand(InvocationContext ctx, RemoveDataCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -93,7 +93,7 @@
    }
 
    @Override
-   public final Object handleEvictFqnCommand(InvocationContext ctx, EvictCommand command) throws Throwable
+   public final Object visitEvictFqnCommand(InvocationContext ctx, EvictCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -108,7 +108,7 @@
    }
 
    @Override
-   public final Object handleInvalidateCommand(InvocationContext ctx, InvalidateCommand command) throws Throwable
+   public final Object visitInvalidateCommand(InvocationContext ctx, InvalidateCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -123,7 +123,7 @@
    }
 
    @Override
-   public final Object handleRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
+   public final Object visitRemoveKeyCommand(InvocationContext ctx, RemoveKeyCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -138,7 +138,7 @@
    }
 
    @Override
-   public final Object handleGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
+   public final Object visitGetDataMapCommand(InvocationContext ctx, GetDataMapCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -153,7 +153,7 @@
    }
 
    @Override
-   public final Object handleExistsNodeCommand(InvocationContext ctx, RemoteExistsCommand command) throws Throwable
+   public final Object visitExistsNodeCommand(InvocationContext ctx, RemoteExistsCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -168,7 +168,7 @@
    }
 
    @Override
-   public final Object handleGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
+   public final Object visitGetKeyValueCommand(InvocationContext ctx, GetKeyValueCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -183,7 +183,7 @@
    }
 
    @Override
-   public final Object handleGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
+   public final Object visitGetNodeCommand(InvocationContext ctx, GetNodeCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -198,7 +198,7 @@
    }
 
    @Override
-   public final Object handleGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
+   public final Object visitGetKeysCommand(InvocationContext ctx, GetKeysCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -213,7 +213,7 @@
    }
 
    @Override
-   public final Object handleGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
+   public final Object visitGetChildrenNamesCommand(InvocationContext ctx, GetChildrenNamesCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -228,7 +228,7 @@
    }
 
    @Override
-   public final Object handleMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
+   public final Object visitMoveCommand(InvocationContext ctx, MoveCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -243,7 +243,7 @@
    }
 
    @Override
-   public final Object handleGravitateDataCommand(InvocationContext ctx, GravitateDataCommand command) throws Throwable
+   public final Object visitGravitateDataCommand(InvocationContext ctx, GravitateDataCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -258,7 +258,7 @@
    }
 
    @Override
-   public final Object handlePrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
+   public final Object visitPrepareCommand(InvocationContext ctx, PrepareCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -273,7 +273,7 @@
    }
 
    @Override
-   public final Object handleRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
+   public final Object visitRollbackCommand(InvocationContext ctx, RollbackCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -288,7 +288,7 @@
    }
 
    @Override
-   public final Object handleCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
+   public final Object visitCommitCommand(InvocationContext ctx, CommitCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {
@@ -303,7 +303,7 @@
    }
 
    @Override
-   public final Object handleOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
+   public final Object visitOptimisticPrepareCommand(InvocationContext ctx, OptimisticPrepareCommand command) throws Throwable
    {
       if (skipInterception(ctx))
       {

Modified: core/trunk/src/test/java/org/jboss/cache/marshall/MarshalledValueTest.java
===================================================================
--- core/trunk/src/test/java/org/jboss/cache/marshall/MarshalledValueTest.java	2008-04-28 15:46:13 UTC (rev 5727)
+++ core/trunk/src/test/java/org/jboss/cache/marshall/MarshalledValueTest.java	2008-04-28 15:46:50 UTC (rev 5728)
@@ -325,7 +325,7 @@
       int invocationCount = 0;
 
 
-      public Object handlePutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
+      public Object visitPutDataMapCommand(InvocationContext ctx, PutDataMapCommand command) throws Throwable
       {
          invocationCount++;
          Object retval = invokeNextInterceptor(ctx, command);
@@ -333,7 +333,7 @@
          return retval;
       }
 
-      public Object handlePutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
+      public Object visitPutKeyValueCommand(InvocationContext ctx, PutKeyValueCommand command) throws Throwable
       {
          invocationCount++;
          if (command.getKey() instanceof MarshalledValue)




More information about the jbosscache-commits mailing list