You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by nn...@apache.org on 2018/11/29 22:28:12 UTC
[geode] branch develop updated: GEODE-6094: Fixed string format
arguments (#2909)
This is an automated email from the ASF dual-hosted git repository.
nnag pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git
The following commit(s) were added to refs/heads/develop by this push:
new 287114c GEODE-6094: Fixed string format arguments (#2909)
287114c is described below
commit 287114c40258f10a92b59aa14e98f74a46356eb6
Author: Nabarun Nag <na...@users.noreply.github.com>
AuthorDate: Thu Nov 29 14:28:03 2018 -0800
GEODE-6094: Fixed string format arguments (#2909)
---
.../src/main/java/org/apache/geode/distributed/LocatorLauncher.java | 2 +-
.../src/main/java/org/apache/geode/distributed/ServerLauncher.java | 2 +-
.../cache/partitioned/rebalance/model/PartitionedRegionLoadModel.java | 2 +-
.../geode/rest/internal/web/controllers/AbstractBaseController.java | 2 +-
4 files changed, 4 insertions(+), 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
index a327eee..d56cfc1 100644
--- 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 class LocatorLauncher extends AbstractLauncher<String> {
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
index 5d8a572..d677399 100755
--- 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 class ServerLauncher extends AbstractLauncher<String> {
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/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/rebalance/model/PartitionedRegionLoadModel.java b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/rebalance/model/PartitionedRegionLoadModel.java
index dc35e04..6813812 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/rebalance/model/PartitionedRegionLoadModel.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/rebalance/model/PartitionedRegionLoadModel.java
@@ -819,7 +819,7 @@ public class PartitionedRegionLoadModel {
}
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/geode-web-api/src/main/java/org/apache/geode/rest/internal/web/controllers/AbstractBaseController.java b/geode-web-api/src/main/java/org/apache/geode/rest/internal/web/controllers/AbstractBaseController.java
index b98bace..2466430 100644
--- a/geode-web-api/src/main/java/org/apache/geode/rest/internal/web/controllers/AbstractBaseController.java
+++ b/geode-web-api/src/main/java/org/apache/geode/rest/internal/web/controllers/AbstractBaseController.java
@@ -404,7 +404,7 @@ public abstract class AbstractBaseController implements InitializingBean {
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) {