Repository: kudu
Updated Branches:
  refs/heads/master 74065d1ca -> 4dbfb6247


Allow multi-master AlterTable tests in TestKuduTable.java

Now that AlterTable is supported with multimaster, and the tests use
multiple masters, we can enable these tests so they run with multiple
masters.

Change-Id: Ieebc0602376be921ec5d7c3f4e5d88b7f4e9356d
Reviewed-on: http://gerrit.cloudera.org:8080/4406
Tested-by: Kudu Jenkins
Reviewed-by: Jean-Daniel Cryans <[email protected]>


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/4dbfb624
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/4dbfb624
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/4dbfb624

Branch: refs/heads/master
Commit: 4dbfb6247553d3396c51a0a8be8df9a0cadec129
Parents: 74065d1
Author: Will Berkeley <[email protected]>
Authored: Tue Sep 13 18:37:43 2016 -0400
Committer: Jean-Daniel Cryans <[email protected]>
Committed: Wed Sep 14 03:42:21 2016 +0000

----------------------------------------------------------------------
 .../src/test/java/org/apache/kudu/client/TestKuduTable.java     | 5 -----
 1 file changed, 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/4dbfb624/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduTable.java
----------------------------------------------------------------------
diff --git 
a/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduTable.java 
b/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduTable.java
index b2a4d01..4f0079a 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduTable.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduTable.java
@@ -117,11 +117,6 @@ public class TestKuduTable extends BaseKuduTest {
   private void submitAlterAndCheck(AlterTableOptions ato,
                                          String tableToAlter, String 
tableToCheck) throws
       Exception {
-    if (masterHostPorts.size() > 1) {
-      LOG.info("Alter table is not yet supported with multiple masters. 
Specify " +
-          "-DnumMasters=1 on the command line to start a single-master cluster 
to run this test.");
-      return;
-    }
     AlterTableResponse alterResponse = syncClient.alterTable(tableToAlter, 
ato);
     boolean done  = syncClient.isAlterTableDone(tableToCheck);
     assertTrue(done);

Reply via email to