You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by ji...@apache.org on 2017/06/06 20:12:50 UTC

[2/2] geode git commit: GEODE-3033: Fixing NPE when jarFileNames is null in ClusterConfigurationLoader

GEODE-3033: Fixing NPE when jarFileNames is null in ClusterConfigurationLoader

* this closes #561

(cherry picked from commit f8c5048)


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

Branch: refs/heads/release/1.2.0
Commit: d4f7a2808f4814217c00d396cb1d3037b81f6def
Parents: a91bbf4
Author: YehEmily <em...@gmail.com>
Authored: Mon Jun 5 14:21:31 2017 -0700
Committer: Jinmei Liao <ji...@pivotal.io>
Committed: Tue Jun 6 13:12:00 2017 -0700

----------------------------------------------------------------------
 .../apache/geode/internal/cache/ClusterConfigurationLoader.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/geode/blob/d4f7a280/geode-core/src/main/java/org/apache/geode/internal/cache/ClusterConfigurationLoader.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/ClusterConfigurationLoader.java b/geode-core/src/main/java/org/apache/geode/internal/cache/ClusterConfigurationLoader.java
index 37f2120..4f4881f 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/ClusterConfigurationLoader.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/ClusterConfigurationLoader.java
@@ -72,9 +72,9 @@ public class ClusterConfigurationLoader {
 
     String[] jarFileNames = response.getJarNames();
     byte[][] jarBytes = response.getJars();
-    logger.info("Got response with jars: {}", Stream.of(jarFileNames).collect(joining(",")));
 
     if (jarFileNames != null && jarBytes != null) {
+      logger.info("Got response with jars: {}", Stream.of(jarFileNames).collect(joining(",")));
       JarDeployer jarDeployer = ClassPathLoader.getLatest().getJarDeployer();
       jarDeployer.suspendAll();
       try {