geode-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From dschnei...@apache.org
Subject [06/50] [abbrv] incubator-geode git commit: comments
Date Thu, 02 Jun 2016 00:09:53 GMT
comments


Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/800c9cb0
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/800c9cb0
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/800c9cb0

Branch: refs/heads/feature/GEODE-1464
Commit: 800c9cb0dfb9436d3ed355f305215bc2aeeddc9f
Parents: f50a5f7
Author: Darrel Schneider <dschneider@pivotal.io>
Authored: Wed Jun 1 15:15:50 2016 -0700
Committer: Darrel Schneider <dschneider@pivotal.io>
Committed: Wed Jun 1 15:15:50 2016 -0700

----------------------------------------------------------------------
 .../com/gemstone/gemfire/internal/cache/BucketRegion.java    | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/800c9cb0/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/BucketRegion.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/BucketRegion.java
b/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/BucketRegion.java
index 56135fb..94bedc4 100644
--- a/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/BucketRegion.java
+++ b/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/BucketRegion.java
@@ -1724,10 +1724,8 @@ implements Bucket
       setDeltaIfNeeded(event);
     }
     if (msg != null) {
-      // The primary bucket member which is being modified remotely by a GemFire
+      // The primary bucket member which is being modified remotely by a
       // thread via a received PartitionedMessage
-      //Asif: Some of the adjunct recepients include those members which 
-      // are sqlFabricHub & would need old value along with news
       msg = msg.getMessageForRelayToListeners(event, adjunctRecipients);
       msg.setSender(this.partitionedRegion.getDistributionManager()
           .getDistributionManagerId());
@@ -2026,11 +2024,11 @@ implements Bucket
       this.entries.removeEntry(event.getKey(), re, updateStats) ;      
     }/*else {
       //if event recorded is true, that means as per event tracker entry is in
-      //system. As per sqlfabric, indexes have been updated. What is not done
+      //system. What is not done
       // is basicPutPart2( distribution etc). So we do nothing as PR's re-attempt
       // will do the required basicPutPart2. If we remove the entry here, than 
       //event tracker will not allow re insertion. So either we do nothing or
-      //if we remove ,than we have to update sqlfindexes as well as undo recording
+      //if we remove ,than we have to undo recording
       // of event.
        //TODO:OQL indexes? : Hope they get updated during retry. The issue is that oql indexes
        // get updated after distribute , so it is entirely possible that oql index are 


Mime
View raw message