Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12872 )
Change subject: java: ensure KuduTestHarness or RetryRule in every test ...................................................................... java: ensure KuduTestHarness or RetryRule in every test Now that test reporting is built into the RetryRule, we should ensure that every test uses either RetryRule or KuduTestHarness (which wraps RetryRule). This patch adds RetryRule to all tests that were missing one of the two. Change-Id: I951f9fbb516abdb24a74d5a2acd7e1f1cd8a6fa5 Reviewed-on: http://gerrit.cloudera.org:8080/12872 Tested-by: Adar Dembo <a...@cloudera.com> Reviewed-by: Grant Henke <granthe...@apache.org> --- M java/kudu-client/src/test/java/org/apache/kudu/TestColumnSchema.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestBitSet.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestBloomFilter.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestBytes.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestColumnRangePredicate.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/TestErrorCollector.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduPredicate.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiator.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestOperation.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestPartialRow.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestRemoteTablet.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestRequestTracker.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestRpcTraces.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestServerInfo.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestStatus.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestTableLocationsCache.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestTimeoutTracker.java M java/kudu-client/src/test/java/org/apache/kudu/util/TestAsyncUtil.java M java/kudu-client/src/test/java/org/apache/kudu/util/TestByteVec.java M java/kudu-client/src/test/java/org/apache/kudu/util/TestMurmurHash.java M java/kudu-client/src/test/java/org/apache/kudu/util/TestNetUtil.java M java/kudu-client/src/test/java/org/apache/kudu/util/TestStringUtil.java M java/kudu-client/src/test/java/org/apache/kudu/util/TestTimestampUtil.java M java/kudu-mapreduce/src/test/java/org/apache/kudu/mapreduce/TestJarFinder.java M java/kudu-test-utils/src/test/java/org/apache/kudu/test/cluster/TestKuduBinaryJarExtractor.java M java/kudu-test-utils/src/test/java/org/apache/kudu/test/junit/TestResultReporter.java 27 files changed, 193 insertions(+), 46 deletions(-) Approvals: Adar Dembo: Verified Grant Henke: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/12872 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I951f9fbb516abdb24a74d5a2acd7e1f1cd8a6fa5 Gerrit-Change-Number: 12872 Gerrit-PatchSet: 7 Gerrit-Owner: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>