This is an automated email from the ASF dual-hosted git repository. ctubbsii pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/accumulo.git
The following commit(s) were added to refs/heads/master by this push: new 0027508 Fix MiniAccumuloClusterExistingZooKeepersTest 0027508 is described below commit 00275086824bbca3fc7711bda14201e9b01e212c Author: Christopher Tubbs <ctubb...@apache.org> AuthorDate: Thu Feb 27 19:39:06 2020 -0500 Fix MiniAccumuloClusterExistingZooKeepersTest * Follow up issue from #1531 * Update curator to latest for zookeeper testing server * Disable ZK adminServer in test, because it listens on 8080 by default, which can conflict with other services in the testing environment, and it is not needed --- .../accumulo/minicluster/MiniAccumuloClusterExistingZooKeepersTest.java | 2 ++ pom.xml | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterExistingZooKeepersTest.java b/minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterExistingZooKeepersTest.java index ac24d8f..609dba0 100644 --- a/minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterExistingZooKeepersTest.java +++ b/minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterExistingZooKeepersTest.java @@ -63,6 +63,8 @@ public class MiniAccumuloClusterExistingZooKeepersTest { FileUtils.deleteQuietly(testDir); assertTrue(testDir.mkdir()); + // disable adminServer, which runs on port 8080 by default and we don't need + System.setProperty("zookeeper.admin.enableServer", "false"); zooKeeper = new TestingServer(); MiniAccumuloConfig config = new MiniAccumuloConfig(testDir, SECRET); diff --git a/pom.xml b/pom.xml index 5c2a4a1..e4b1d4e 100644 --- a/pom.xml +++ b/pom.xml @@ -117,7 +117,7 @@ <!-- bouncycastle version for test dependencies --> <bouncycastle.version>1.62</bouncycastle.version> <!-- Curator version --> - <curator.version>2.12.0</curator.version> + <curator.version>4.2.0</curator.version> <!-- relative path for Eclipse format; should override in child modules if necessary --> <eclipseFormatterStyle>${project.parent.basedir}/contrib/Eclipse-Accumulo-Codestyle.xml</eclipseFormatterStyle> <!-- extra release args for testing -->