Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11547 )

Change subject: [test] Move BaseKuduTest to a Junit Rule
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11547/1/java/kudu-client/src/test/java/org/apache/kudu/test/KuduRule.java
File java/kudu-client/src/test/java/org/apache/kudu/test/KuduRule.java:

http://gerrit.cloudera.org:8080/#/c/11547/1/java/kudu-client/src/test/java/org/apache/kudu/test/KuduRule.java@43
PS1, Line 43: public class KuduRule extends ExternalResource {
> I don't think so. Add a class-level comment with sample usage and that shou
Done



--
To view, visit http://gerrit.cloudera.org:8080/11547
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I32c83b47a576377b924ea41dbeaf78ce2b75e4c4
Gerrit-Change-Number: 11547
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Comment-Date: Tue, 02 Oct 2018 18:24:59 +0000
Gerrit-HasComments: Yes

Reply via email to