This is an automated email from the ASF dual-hosted git repository.

eshu11 pushed a commit to branch feature/GEODE-5376
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/feature/GEODE-5376 by this 
push:
     new 1cca59e  fix a review comment.
1cca59e is described below

commit 1cca59ea61e34f068aabaeab322db0a868e05b09
Author: eshu <[email protected]>
AuthorDate: Thu Jul 19 15:46:52 2018 -0700

    fix a review comment.
---
 geode-core/src/main/java/org/apache/geode/internal/cache/TXState.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/geode-core/src/main/java/org/apache/geode/internal/cache/TXState.java 
b/geode-core/src/main/java/org/apache/geode/internal/cache/TXState.java
index 211c801..389d0e7 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/TXState.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/TXState.java
@@ -98,7 +98,8 @@ public class TXState implements TXStateInterface {
    */
   private int modSerialNum;
   private final List<EntryEventImpl> pendingCallbacks = new 
ArrayList<EntryEventImpl>();
-  private volatile boolean beforeCompletionCalled;
+  // Access this variable should be in synchronized block.
+  private boolean beforeCompletionCalled;
 
   // Internal testing hooks
   private Runnable internalAfterReservation;

Reply via email to