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

[2/7] 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


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

Branch: refs/heads/feature/GEODE-3027
Commit: f8c504847279bd71da9e193445a66b58af1aeb4d
Parents: 1e11c5c
Author: YehEmily <em...@gmail.com>
Authored: Mon Jun 5 14:21:31 2017 -0700
Committer: Jinmei Liao <ji...@pivotal.io>
Committed: Tue Jun 6 08:39:02 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/f8c50484/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 {