You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by zh...@apache.org on 2018/01/19 16:10:26 UTC

[geode] branch feature/GEM-883 updated: GEODE-3967: reduce some log

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

zhouxj pushed a commit to branch feature/GEM-883
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/feature/GEM-883 by this push:
     new c1fd7c6  GEODE-3967: reduce some log
c1fd7c6 is described below

commit c1fd7c6ff1b7656d8b1e62e85f08060290576e89
Author: zhouxh <gz...@pivotal.io>
AuthorDate: Fri Jan 19 08:10:05 2018 -0800

    GEODE-3967: reduce some log
---
 .../main/java/org/apache/geode/internal/cache/AbstractRegionMap.java | 3 +--
 .../org/apache/geode/internal/cache/AbstractUpdateOperation.java     | 5 +----
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/AbstractRegionMap.java b/geode-core/src/main/java/org/apache/geode/internal/cache/AbstractRegionMap.java
index fbd3a67..d1f6085 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/AbstractRegionMap.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/AbstractRegionMap.java
@@ -2820,6 +2820,7 @@ public abstract class AbstractRegionMap implements RegionMap {
                 }
                 // overwrite destroyed?
                 if (!continueOverwriteDestroyed(re, event, overwriteDestroyed, ifNew)) {
+                  logger.info("GGG:not continueOverwriteDestroyed");
                   return null;
                 }
                 // check expectedOldValue
@@ -2842,11 +2843,9 @@ public abstract class AbstractRegionMap implements RegionMap {
                                                                         // set
                         || !re.isRemoved() || replaceOnClient) {
                       // update
-                      logger.info("GGG:before updateEntry:" + event);
                       updateEntry(event, requireOldValue, oldValueForDelta, re);
                     } else {
                       // create
-                      logger.info("GGG:before createEntry:" + event);
                       createEntry(event, owner, re);
                     }
                     owner.recordEvent(event);
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/AbstractUpdateOperation.java b/geode-core/src/main/java/org/apache/geode/internal/cache/AbstractUpdateOperation.java
index ad38a94..906a086 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/AbstractUpdateOperation.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/AbstractUpdateOperation.java
@@ -133,7 +133,7 @@ public abstract class AbstractUpdateOperation extends DistributedCacheOperation
             doUpdate = false;
             updated = true;
             if (logger.isDebugEnabled()) {
-              logger.debug("GGG:Created put key " + ev);
+              logger.debug("GGG:Created put key ");
             }
           } else { // already exists. If it was blocked by the DESTROYED token, then
             // do no update.
@@ -143,9 +143,6 @@ public abstract class AbstractUpdateOperation extends DistributedCacheOperation
                     (InternalDistributedMember) ev.getDistributedMember(), ev.getVersionTag());
               }
               doUpdate = false;
-              logger.info("GGG:oldValueIsDestroyedToken,doUpdate=false:" + ev);
-            } else {
-              logger.info("GGG:oldValueIsDestroyedToken false, doUpdate=true:" + ev);
             }
           }
         } finally {

-- 
To stop receiving notification emails like this one, please contact
['"commits@geode.apache.org" <co...@geode.apache.org>'].