Repository: hbase
Updated Branches:
  refs/heads/master fe9180d79 -> 07b8a7b26


HBASE-13524 TestReplicationAdmin fails on JDK 1.8


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/07b8a7b2
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/07b8a7b2
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/07b8a7b2

Branch: refs/heads/master
Commit: 07b8a7b264bc6d91fa63243b2e9cd5527176a6f5
Parents: fe9180d
Author: Elliott Clark <ecl...@apache.org>
Authored: Tue Apr 21 11:13:52 2015 -0700
Committer: Elliott Clark <ecl...@apache.org>
Committed: Tue Apr 21 17:06:10 2015 -0700

----------------------------------------------------------------------
 .../client/replication/TestReplicationAdmin.java    | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/07b8a7b2/hbase-server/src/test/java/org/apache/hadoop/hbase/client/replication/TestReplicationAdmin.java
----------------------------------------------------------------------
diff --git 
a/hbase-server/src/test/java/org/apache/hadoop/hbase/client/replication/TestReplicationAdmin.java
 
b/hbase-server/src/test/java/org/apache/hadoop/hbase/client/replication/TestReplicationAdmin.java
index 4db646e..d1083b8 100644
--- 
a/hbase-server/src/test/java/org/apache/hadoop/hbase/client/replication/TestReplicationAdmin.java
+++ 
b/hbase-server/src/test/java/org/apache/hadoop/hbase/client/replication/TestReplicationAdmin.java
@@ -167,11 +167,23 @@ public class TestReplicationAdmin {
 
     // append table t2 to replication
     admin.appendPeerTableCFs(ID_ONE, "t2");
-    assertEquals("t2;t1", admin.getPeerTableCFs(ID_ONE));
+    String peerTablesOne = admin.getPeerTableCFs(ID_ONE);
+
+    // Different jdk's return different sort order for the tables. ( Not sure 
on why exactly )
+    //
+    // So instead of asserting that the string is exactly we
+    // assert that the string contains all tables and the needed separator.
+    assertTrue("Should contain t1", peerTablesOne.contains("t1"));
+    assertTrue("Should contain t2", peerTablesOne.contains("t2"));
+    assertTrue("Should contain ; as the seperator", 
peerTablesOne.contains(";"));
 
     // append table column family: f1 of t3 to replication
     admin.appendPeerTableCFs(ID_ONE, "t3:f1");
-    assertEquals("t3:f1;t2;t1", admin.getPeerTableCFs(ID_ONE));
+    String peerTablesTwo = admin.getPeerTableCFs(ID_ONE);
+    assertTrue("Should contain t1", peerTablesTwo.contains("t1"));
+    assertTrue("Should contain t2", peerTablesTwo.contains("t2"));
+    assertTrue("Should contain t3:f1", peerTablesTwo.contains("t3:f1"));
+    assertTrue("Should contain ; as the seperator", 
peerTablesTwo.contains(";"));
     admin.removePeer(ID_ONE);
   }
 

Reply via email to