diff --git a/hadoop-common-project/hadoop-common/CHANGES.txt b/hadoop-common-project/hadoop-common/CHANGES.txt index 61d25b45a6..16d23c8095 100644 --- a/hadoop-common-project/hadoop-common/CHANGES.txt +++ b/hadoop-common-project/hadoop-common/CHANGES.txt @@ -170,6 +170,9 @@ Trunk (Unreleased) HADOOP-8608. Add Configuration API for parsing time durations. (cdouglas) + HADOOP-9540. Expose the InMemoryS3 and S3N FilesystemStores implementations + for Unit testing. (Hari via stevel) + BUG FIXES HADOOP-9451. Fault single-layer config if node group topology is enabled. @@ -548,8 +551,9 @@ Trunk (Unreleased) HADOOP-9483. winutils support for readlink command. (Arpit Agarwal via suresh) - HADOOP-9540. Expose the InMemoryS3 and S3N FilesystemStores implementations for Unit testing. - (Hari via stevel) + HADOOP-9556. disable HA tests on Windows that fail due to ZooKeeper client + connection management bug. (Chris Nauroth via suresh) + Release 2.0.5-beta - UNRELEASED diff --git a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/ha/TestActiveStandbyElectorRealZK.java b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/ha/TestActiveStandbyElectorRealZK.java index d51d5fa700..7aee139a8e 100644 --- a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/ha/TestActiveStandbyElectorRealZK.java +++ b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/ha/TestActiveStandbyElectorRealZK.java @@ -20,6 +20,7 @@ package org.apache.hadoop.ha; import static org.junit.Assert.assertFalse; import static org.junit.Assert.assertTrue; +import static org.junit.Assume.assumeTrue; import java.util.Collections; import java.util.UUID; @@ -28,6 +29,7 @@ import org.apache.commons.logging.impl.Log4JLogger; import org.apache.hadoop.ha.ActiveStandbyElector.ActiveStandbyElectorCallback; import org.apache.hadoop.ha.ActiveStandbyElector.State; import org.apache.hadoop.ha.HAZKUtil.ZKAuthInfo; +import org.apache.hadoop.util.Shell; import org.apache.log4j.Level; import org.apache.zookeeper.ZooDefs.Ids; import org.apache.zookeeper.server.ZooKeeperServer; @@ -59,6 +61,8 @@ public class TestActiveStandbyElectorRealZK extends ClientBaseWithFixes { @Override public void setUp() throws Exception { + // skip tests on Windows until after resolution of ZooKeeper client bug + assumeTrue(!Shell.WINDOWS); super.setUp(); zkServer = getServer(serverFactory); diff --git a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/ha/TestZKFailoverController.java b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/ha/TestZKFailoverController.java index ed44d8a033..59f693a9c1 100644 --- a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/ha/TestZKFailoverController.java +++ b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/ha/TestZKFailoverController.java @@ -18,6 +18,7 @@ package org.apache.hadoop.ha; import static org.junit.Assert.*; +import static org.junit.Assume.assumeTrue; import java.security.NoSuchAlgorithmException; @@ -28,6 +29,7 @@ import org.apache.hadoop.ha.HAServiceProtocol.StateChangeRequestInfo; import org.apache.hadoop.ha.HealthMonitor.State; import org.apache.hadoop.ha.MiniZKFCCluster.DummyZKFC; import org.apache.hadoop.test.GenericTestUtils; +import org.apache.hadoop.util.Shell; import org.apache.hadoop.util.Time; import org.apache.log4j.Level; import org.apache.zookeeper.KeeperException; @@ -66,6 +68,8 @@ public class TestZKFailoverController extends ClientBaseWithFixes { @Before public void setupConfAndServices() { + // skip tests on Windows until after resolution of ZooKeeper client bug + assumeTrue(!Shell.WINDOWS); conf = new Configuration(); conf.set(ZKFailoverController.ZK_ACL_KEY, TEST_ACL); conf.set(ZKFailoverController.ZK_AUTH_KEY, TEST_AUTH_GOOD); diff --git a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/ha/TestZKFailoverControllerStress.java b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/ha/TestZKFailoverControllerStress.java index e1113c4e80..95874c42df 100644 --- a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/ha/TestZKFailoverControllerStress.java +++ b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/ha/TestZKFailoverControllerStress.java @@ -17,10 +17,13 @@ */ package org.apache.hadoop.ha; +import static org.junit.Assume.assumeTrue; + import java.util.Random; import org.apache.hadoop.conf.Configuration; +import org.apache.hadoop.util.Shell; import org.apache.hadoop.util.Time; import org.junit.After; import org.junit.Before; @@ -45,6 +48,8 @@ public class TestZKFailoverControllerStress extends ClientBaseWithFixes { @Before public void setupConfAndServices() throws Exception { + // skip tests on Windows until after resolution of ZooKeeper client bug + assumeTrue(!Shell.WINDOWS); conf = new Configuration(); conf.set(ZKFailoverController.ZK_QUORUM_KEY, hostPort); this.cluster = new MiniZKFCCluster(conf, getServer(serverFactory)); @@ -52,7 +57,9 @@ public class TestZKFailoverControllerStress extends ClientBaseWithFixes { @After public void stopCluster() throws Exception { - cluster.stop(); + if (cluster != null) { + cluster.stop(); + } } /**