Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/11418


Change subject: WIP: Breakout the Java MiniKuduCluster
......................................................................

WIP: Breakout the Java MiniKuduCluster

This change will simplify using the Java
MiniKuduCluster outside of the Kudu project itself.
This makes integration testing with Kudu more
straightforward by providing a common base
test class for testing.
Additionally some of the test utilities are broken out
to simplify inter-module dependencies.

Follow on work could include more client setup
and testing utilities.

The patch:
- Breaks out the compile protobuf protocol classes
   into its own module/jar because they are used
   directly in the MiniKuduCluster
- Breaks out the MiniKuduCluster and a few supporting
   classes into its own module/jar.
- Adjusts the binary locating logic to be less fragile and
   externally useful.
- Adjust the Maven and Gradle build to define the
   `kuduBinDir` system property. This also fixes the
   issue where Maven tests would fail to locate the
   binary because they were using jars installed into
   the local repository.
- Removed usage of Guava HostAndPort class. It is
   marked as Beta and because we shade and relocate
   Guava it should be privately consumed only.
- Marks the InterfaceAudience and InterfaceStability
   of testing utilities.
- Removed direct usage of the MiniKuduCluster in our
   tests in favor of the KuduClusterTest base testing
   methods.

TODO:
- Documentation (readme and dev docs)

Change-Id: I4f39bc012a19fea5a57e18bc12192e1ea859feba
---
M java/README.adoc
M java/buildSrc/src/main/groovy/org/apache/kudu/gradle/DistTestTask.java
M java/gradle/tests.gradle
M java/kudu-backup/build.gradle
M java/kudu-backup/pom.xml
M java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestKuduBackup.scala
M java/kudu-client-tools/build.gradle
M java/kudu-client-tools/pom.xml
M java/kudu-client/build.gradle
M java/kudu-client/pom.xml
M java/kudu-client/src/test/java/org/apache/kudu/client/BaseKuduTest.java
M 
java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduSession.java
M 
java/kudu-client/src/test/java/org/apache/kudu/client/TestAuthnTokenReacquire.java
M 
java/kudu-client/src/test/java/org/apache/kudu/client/TestAuthnTokenReacquireOpen.java
M 
java/kudu-client/src/test/java/org/apache/kudu/client/TestClientFailoverSupport.java
M 
java/kudu-client/src/test/java/org/apache/kudu/client/TestConnectToCluster.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestConnectionCache.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestHandleTooBusy.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestHybridTime.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClient.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestMasterFailover.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/TestNegotiation.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java
M 
java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurityContextRealUser.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestServerInfo.java
D java/kudu-client/src/test/java/org/apache/kudu/client/TestTestUtils.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestUtils.java
M java/kudu-client/src/test/java/org/apache/kudu/util/CapturingLogAppender.java
M java/kudu-client/src/test/java/org/apache/kudu/util/ClientTestUtil.java
M java/kudu-client/src/test/java/org/apache/kudu/util/TestAsyncUtil.java
M java/kudu-flume-sink/build.gradle
M java/kudu-flume-sink/pom.xml
M 
java/kudu-flume-sink/src/test/java/org/apache/kudu/flume/sink/AvroKuduOperationsProducerTest.java
M java/kudu-hive/build.gradle
M java/kudu-hive/pom.xml
M java/kudu-mapreduce/build.gradle
M java/kudu-mapreduce/pom.xml
A java/kudu-protocol/build.gradle
A java/kudu-protocol/pom.xml
M java/kudu-spark-tools/build.gradle
M java/kudu-spark-tools/pom.xml
M java/kudu-spark/build.gradle
M java/kudu-spark/pom.xml
M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduTestSuite.scala
A java/kudu-test-utils/build.gradle
A java/kudu-test-utils/pom.xml
R java/kudu-test-utils/src/main/java/org/apache/kudu/junit/AssertHelpers.java
R java/kudu-test-utils/src/main/java/org/apache/kudu/junit/RetryRule.java
A java/kudu-test-utils/src/main/java/org/apache/kudu/test/RandomUtils.java
A 
java/kudu-test-utils/src/main/java/org/apache/kudu/test/cluster/KuduBinaryLocator.java
A 
java/kudu-test-utils/src/main/java/org/apache/kudu/test/cluster/KuduClusterTest.java
R 
java/kudu-test-utils/src/main/java/org/apache/kudu/test/cluster/MiniKuduCluster.java
R java/kudu-test-utils/src/main/java/org/apache/kudu/test/network/FakeDNS.java
M java/pom.xml
M java/settings.gradle
57 files changed, 995 insertions(+), 636 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/18/11418/1
--
To view, visit http://gerrit.cloudera.org:8080/11418
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4f39bc012a19fea5a57e18bc12192e1ea859feba
Gerrit-Change-Number: 11418
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <granthe...@apache.org>

Reply via email to