GEODE-6094: Fixed string format arguments (apache#2909)

adl-oss · Nov 29, 2018 · 287114c · 287114c
1 parent 944951f
commit 287114c
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/geode-core/src/main/java/org/apache/geode/distributed/LocatorLauncher.java b/geode-core/src/main/java/org/apache/geode/distributed/LocatorLauncher.java
@@ -667,7 +667,7 @@ public LocatorState start() {
           ProcessLauncherContext.remove();
         }
 
-        debug("Running Locator on (%1$s) in (%2$s) as (%2$s)...", getId(), getWorkingDirectory(),
+        debug("Running Locator on (%1$s) in (%2$s) as (%3$s)...", getId(), getWorkingDirectory(),
             getMember());
         running.set(true);
 

diff --git a/geode-core/src/main/java/org/apache/geode/distributed/ServerLauncher.java b/geode-core/src/main/java/org/apache/geode/distributed/ServerLauncher.java
@@ -800,7 +800,7 @@ public ServerState start() {
           ProcessLauncherContext.remove();
         }
 
-        debug("Running Server on (%1$s) in (%2$s) as (%2$s)...", getId(), getWorkingDirectory(),
+        debug("Running Server on (%1$s) in (%2$s) as (%3$s)...", getId(), getWorkingDirectory(),
             getMember());
         this.running.set(true);
 

diff --git a/...g/apache/geode/internal/cache/partitioned/rebalance/model/PartitionedRegionLoadModel.java b/...g/apache/geode/internal/cache/partitioned/rebalance/model/PartitionedRegionLoadModel.java
@@ -819,7 +819,7 @@ public String toString() {
     }
 
     result.append(String.format("\n%" + longestMemberId + "s                            ",
-        "#offline", 0, 0, 0));
+        "#offline"));
     for (Bucket bucket : allBucketIds) {
       result.append(String.format("%4s", bucket.getOfflineMembers().size()));
     }

diff --git a/.../src/main/java/org/apache/geode/rest/internal/web/controllers/AbstractBaseController.java b/.../src/main/java/org/apache/geode/rest/internal/web/controllers/AbstractBaseController.java
@@ -404,7 +404,7 @@ void checkForQueryIdExist(String region, String key) {
 
   Region<String, String> getQueryStore(final String namePath) {
     return ValidationUtils.returnValueThrowOnNull(getCache().getInternalRegion(namePath),
-        new GemfireRestException(String.format("Query store does not exist!", namePath)));
+        new GemfireRestException(String.format("Query store (%1$s) does not exist!", namePath)));
   }
 
   protected String getQueryIdValue(final String regionNamePath, final String key) {