You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by kl...@apache.org on 2017/08/01 21:02:57 UTC

[03/50] [abbrv] geode git commit: GEODE-2986: Remove redundant log message

GEODE-2986: Remove redundant log message

Just log message if "notExecutedMembers" is not null and not empty.

This closes #535


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

Branch: refs/heads/feature/GEODE-3299
Commit: 82894a67ba5e875ff9e6a187d5102d91ca8ab738
Parents: 6267efe
Author: Akihiro Kitada <ak...@akitada-mcbk13.local>
Authored: Thu May 25 11:16:08 2017 +0900
Committer: Darrel Schneider <ds...@pivotal.io>
Committed: Tue Jul 25 16:02:45 2017 -0700

----------------------------------------------------------------------
 .../internal/cli/commands/DiskStoreCommands.java          | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/geode/blob/82894a67/geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DiskStoreCommands.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DiskStoreCommands.java b/geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DiskStoreCommands.java
index 96aa95e..84326c0 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DiskStoreCommands.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DiskStoreCommands.java
@@ -475,10 +475,12 @@ public class DiskStoreCommands implements GfshCommand {
               memberCompactInfo.clear();
             }
             String notExecutedMembers = CompactRequest.getNotExecutedMembers();
-            LogWrapper.getInstance()
-                .info("compact disk-store \"" + diskStoreName
-                    + "\" message was scheduled to be sent to but was not send to "
-                    + notExecutedMembers);
+            if (notExecutedMembers != null && !notExecutedMembers.isEmpty()) {
+              LogWrapper.getInstance()
+                  .info("compact disk-store \"" + diskStoreName
+                      + "\" message was scheduled to be sent to but was not send to "
+                      + notExecutedMembers);
+            }
           }
 
           // If compaction happened at all, then prepare the summary