commit 57582dcee0f72a64783873f09a4babadadb7a95f
author Young-Seok <kisskys@gmail.com> Thu Sep 19 17:44:34 2013 -0700
committer Young-Seok <kisskys@gmail.com> Thu Sep 19 17:44:34 2013 -0700
tree 57ad4a957cb47d0f5eb51b146bab878c2d217605
parent 2ebc052d972186d377ee4378e790a03c8f477563

addressed code review comments

asterix-common/src/main/java/edu/uci/ics/asterix/common/context/PrimaryIndexOperationTracker.java

diff --git a/asterix-common/src/main/java/edu/uci/ics/asterix/common/context/PrimaryIndexOperationTracker.java b/asterix-common/src/main/java/edu/uci/ics/asterix/common/context/PrimaryIndexOperationTracker.java
index 5adb361..ef58937 100644
--- a/asterix-common/src/main/java/edu/uci/ics/asterix/common/context/PrimaryIndexOperationTracker.java
+++ b/asterix-common/src/main/java/edu/uci/ics/asterix/common/context/PrimaryIndexOperationTracker.java
@@ -100,14 +100,16 @@
     }
 
     private void incrementNumActiveOperations(IModificationOperationCallback modificationCallback) {
-        if (modificationCallback != null && modificationCallback != NoOpOperationCallback.INSTANCE) {
+        //modificationCallback can be NoOpOperationCallback when redo/undo operations are executed. 
+        if (modificationCallback != NoOpOperationCallback.INSTANCE) {
             numActiveOperations.incrementAndGet();
             ((AbstractOperationCallback) modificationCallback).incrementLocalNumActiveOperations();
         }
     }
 
     private void decrementNumActiveOperations(IModificationOperationCallback modificationCallback) {
-        if (modificationCallback != null && modificationCallback != NoOpOperationCallback.INSTANCE) {
+        //modificationCallback can be NoOpOperationCallback when redo/undo operations are executed.
+        if (modificationCallback != NoOpOperationCallback.INSTANCE) {
             numActiveOperations.decrementAndGet();
             ((AbstractOperationCallback) modificationCallback).decrementLocalNumActiveOperations();
         }