Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/2828
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, 235 insertions(+), 66 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/28/2828/1 -- To view, visit http://gerrit.cloudera.org:8080/2828 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I065917cd219e1dd8fa93fed9ca36d58a3372935d Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel Cryans
