Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8194 )
Change subject: java: replace bespoke minicluster implementation with control shell ...................................................................... java: replace bespoke minicluster implementation with control shell This patch replaces the Java client's bespoke MiniKuduCluster and MiniKdc implementations with appropriate calls to the new CLI control shell. The shell can communicate either in protobuf or JSON; I've opted to use the former here since the Java client already understands protobuf, and doing so yields more type safety. Change-Id: Ia6340dadd6ff0397fffa23388a8cbbca3e26a618 Reviewed-on: http://gerrit.cloudera.org:8080/8194 Reviewed-by: Dan Burkert <[email protected]> Tested-by: Kudu Jenkins --- M java/kudu-client/src/test/java/org/apache/kudu/client/BaseKuduTest.java D java/kudu-client/src/test/java/org/apache/kudu/client/MiniKdc.java M java/kudu-client/src/test/java/org/apache/kudu/client/MiniKuduCluster.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestClientFailoverSupport.java D java/kudu-client/src/test/java/org/apache/kudu/client/TestMiniKdc.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestMiniKuduCluster.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestMultipleLeaderFailover.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestUtils.java D java/kudu-client/src/test/resources/flags M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/TestContext.scala 10 files changed, 401 insertions(+), 1,235 deletions(-) Approvals: Dan Burkert: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8194 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ia6340dadd6ff0397fffa23388a8cbbca3e26a618 Gerrit-Change-Number: 8194 Gerrit-PatchSet: 9 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins
