Hello Dan Burkert, Jean-Daniel Cryans,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/3654

to look at the new patch set (#5).

Change subject: [java-client] Re-enable multi-master tests
......................................................................

[java-client] Re-enable multi-master tests

This patch makes TestMasterFailover useful again. It also adds the killing of 
masters
to ITClient. Finally, it sets the raft heartbeat lower so that we don't wait 
1.5s for
leader elections.

TestGetMasterRegistrationReceived was added since a previous version of this 
patch
encountered a bug that such a simple unit test can detect.

Change-Id: Ia1051222738c84ef3d3e1a33b4981bc9454b7972
---
M 
java/kudu-client/src/main/java/org/apache/kudu/client/GetMasterRegistrationReceived.java
M java/kudu-client/src/test/java/org/apache/kudu/client/BaseKuduTest.java
M java/kudu-client/src/test/java/org/apache/kudu/client/ITClient.java
M java/kudu-client/src/test/java/org/apache/kudu/client/MiniKuduCluster.java
A 
java/kudu-client/src/test/java/org/apache/kudu/client/TestGetMasterRegistrationReceived.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestMasterFailover.java
6 files changed, 252 insertions(+), 34 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/54/3654/5
-- 
To view, visit http://gerrit.cloudera.org:8080/3654
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia1051222738c84ef3d3e1a33b4981bc9454b7972
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jean-Daniel Cryans <jdcry...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <d...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to