You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by kl...@apache.org on 2016/05/05 16:45:45 UTC
incubator-geode git commit: Fix NullPointerException
Repository: incubator-geode
Updated Branches:
refs/heads/feature/GEODE-1255 1e3442222 -> 0b9aa25da
Fix NullPointerException
Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/0b9aa25d
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/0b9aa25d
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/0b9aa25d
Branch: refs/heads/feature/GEODE-1255
Commit: 0b9aa25dab0d5eb7f595f75c33ec3822810ae734
Parents: 1e34422
Author: Kirk Lund <kl...@apache.org>
Authored: Thu May 5 09:45:32 2016 -0700
Committer: Kirk Lund <kl...@apache.org>
Committed: Thu May 5 09:45:32 2016 -0700
----------------------------------------------------------------------
.../SharedConfigurationEndToEndDUnitTest.java | 12 +++++++++---
1 file changed, 9 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/0b9aa25d/geode-assembly/src/test/java/com/gemstone/gemfire/management/internal/configuration/SharedConfigurationEndToEndDUnitTest.java
----------------------------------------------------------------------
diff --git a/geode-assembly/src/test/java/com/gemstone/gemfire/management/internal/configuration/SharedConfigurationEndToEndDUnitTest.java b/geode-assembly/src/test/java/com/gemstone/gemfire/management/internal/configuration/SharedConfigurationEndToEndDUnitTest.java
index 200c379..caee9ea 100644
--- a/geode-assembly/src/test/java/com/gemstone/gemfire/management/internal/configuration/SharedConfigurationEndToEndDUnitTest.java
+++ b/geode-assembly/src/test/java/com/gemstone/gemfire/management/internal/configuration/SharedConfigurationEndToEndDUnitTest.java
@@ -71,8 +71,8 @@ public class SharedConfigurationEndToEndDUnitTest extends CliCommandTestBase {
private static final String REGION2 = "R2";
private static final String INDEX1 = "ID1";
- private static Set<String> serverNames = new HashSet<>();
- private static Set<String> jarFileNames = new HashSet<>();
+ private static Set<String> serverNames;
+ private static Set<String> jarFileNames;
private transient ClassBuilder classBuilder;
private transient String jmxHost;
@@ -86,6 +86,9 @@ public class SharedConfigurationEndToEndDUnitTest extends CliCommandTestBase {
addIgnoredException("EntryDestroyedException");
+ serverNames = new HashSet<>();
+ jarFileNames = new HashSet<>();
+
this.classBuilder = new ClassBuilder();
Object[] result = setup();
@@ -104,6 +107,9 @@ public class SharedConfigurationEndToEndDUnitTest extends CliCommandTestBase {
serverNames.clear();
jarFileNames.clear();
+
+ serverNames = null;
+ jarFileNames = null;
}
@Test
@@ -406,6 +412,7 @@ public class SharedConfigurationEndToEndDUnitTest extends CliCommandTestBase {
});
locatorAndMgr.invoke(SharedConfigurationTestUtils.cleanupLocator);
+
//Clean up the directories
if (!serverNames.isEmpty()) {
for (String serverName : serverNames) {
@@ -415,7 +422,6 @@ public class SharedConfigurationEndToEndDUnitTest extends CliCommandTestBase {
}
}
serverNames.clear();
- serverNames = null;
}
private void verifyRegionCreateOnAllMembers(final String regionName) {