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 2019/01/18 23:14:41 UTC

[geode] branch develop updated: GEODE-6232: Disable JMX in PersistentPartitionedRegionRegressionTest

This is an automated email from the ASF dual-hosted git repository.

klund 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 14359cc  GEODE-6232: Disable JMX in PersistentPartitionedRegionRegressionTest
14359cc is described below

commit 14359cca827f94968bc0997240110343a56f395e
Author: Kirk Lund <kl...@apache.org>
AuthorDate: Thu Jan 17 17:22:05 2019 -0800

    GEODE-6232: Disable JMX in PersistentPartitionedRegionRegressionTest
---
 .../PersistentPartitionedRegionRegressionTest.java            | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/geode-core/src/distributedTest/java/org/apache/geode/internal/cache/partitioned/PersistentPartitionedRegionRegressionTest.java b/geode-core/src/distributedTest/java/org/apache/geode/internal/cache/partitioned/PersistentPartitionedRegionRegressionTest.java
index 466f871..a353ce6 100644
--- a/geode-core/src/distributedTest/java/org/apache/geode/internal/cache/partitioned/PersistentPartitionedRegionRegressionTest.java
+++ b/geode-core/src/distributedTest/java/org/apache/geode/internal/cache/partitioned/PersistentPartitionedRegionRegressionTest.java
@@ -17,6 +17,7 @@ package org.apache.geode.internal.cache.partitioned;
 import static java.util.concurrent.TimeUnit.MINUTES;
 import static java.util.concurrent.TimeUnit.SECONDS;
 import static org.apache.geode.cache.RegionShortcut.PARTITION_PERSISTENT;
+import static org.apache.geode.distributed.ConfigurationProperties.DISABLE_JMX;
 import static org.apache.geode.test.awaitility.GeodeAwaitility.await;
 import static org.apache.geode.test.dunit.Disconnect.disconnectFromDS;
 import static org.apache.geode.test.dunit.IgnoredException.addIgnoredException;
@@ -26,6 +27,7 @@ import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
 import java.io.Serializable;
+import java.util.Properties;
 import java.util.Set;
 import java.util.TreeSet;
 import java.util.concurrent.CountDownLatch;
@@ -587,8 +589,15 @@ public class PersistentPartitionedRegionRegressionTest implements Serializable {
     regionFactory.create(partitionedRegionName);
   }
 
+  /**
+   * Prevent GEODE-6232 by disabling JMX which is not needed in this test.
+   */
   private InternalCache getCache() {
-    return cacheRule.getOrCreateCache();
+    Properties config = new Properties();
+    config.setProperty(DISABLE_JMX, "true");
+    InternalCache cache = cacheRule.getOrCreateCache(config);
+    assertThat(cache.getInternalDistributedSystem().getResourceListeners()).isEmpty();
+    return cache;
   }
 
   private static class TestCustomExpiration<K, V> implements CustomExpiry<K, V> {