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

[3/4] 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/c2fe502b
Tree: http://git-wip-us.apache.org/repos/asf/geode/tree/c2fe502b
Diff: http://git-wip-us.apache.org/repos/asf/geode/diff/c2fe502b

Branch: refs/heads/feature/GEODE-3023
Commit: c2fe502b0e4b8368323b2fbcd2f6abed60b5e18f
Parents: 7670525
Author: YehEmily <em...@gmail.com>
Authored: Mon Jun 5 14:21:31 2017 -0700
Committer: Udo Kohlmeyer <uk...@pivotal.io>
Committed: Tue Jun 6 10:57:19 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/c2fe502b/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 {