Hello Will Berkeley, Mike Percy, Grant Henke,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/12872
to review the following change.
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
---
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(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/72/12872/1
--
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: newchange
Gerrit-Change-Id: I951f9fbb516abdb24a74d5a2acd7e1f1cd8a6fa5
Gerrit-Change-Number: 12872
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Will Berkeley <[email protected]>
