Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/2828
to look at the new patch set (#3).
Change subject: [java client] Add a way to restart processes
......................................................................
[java client] Add a way to restart processes
This patch adds more tracking of the processes started by MiniKuduCluster.
Mainly we needed to record the command lines used to start the masters
and tablet servers.
This patch adds a new unit test for the mini cluster, before we only relied
on the fact that everything was using it so therefore it was tested.
Finally getUniqueLocalhost was moved to TestUtils so that findFreePort
could also use it. The reason is that the processes on OSX were binding
to 0.0.0.0 but findFreePort was using a different hostnames in
ServerSocket and so the behavior was broken. This was noticed when a TS
leaked after a test and new tests were failing because the TS was already
bound to a port we thought to be free.
Change-Id: I065917cd219e1dd8fa93fed9ca36d58a3372935d
---
M java/kudu-client/src/test/java/org/kududb/client/BaseKuduTest.java
M java/kudu-client/src/test/java/org/kududb/client/MiniKuduCluster.java
A java/kudu-client/src/test/java/org/kududb/client/TestMiniKuduCluster.java
M java/kudu-client/src/test/java/org/kududb/client/TestUtils.java
4 files changed, 265 insertions(+), 81 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/28/2828/3
--
To view, visit http://gerrit.cloudera.org:8080/2828
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I065917cd219e1dd8fa93fed9ca36d58a3372935d
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jean-Daniel Cryans
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
