You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@kudu.apache.org by "Hao Hao (Code Review)" <ge...@cloudera.org> on 2020/04/28 23:17:54 UTC

[kudu-CR] [java] use KuduTestHarness in TestSecurity

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15433 )

Change subject: [java] use KuduTestHarness in TestSecurity
......................................................................


Patch Set 4: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/15433/4/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java
File java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java:

http://gerrit.cloudera.org:8080/#/c/15433/4/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java@438
PS4, Line 438: //for (HostAndPort hp : miniCluster.getMasterServers()) {
Nit: remove?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Idb9089a414f5aa2277779c57a07b41405d35d311
Gerrit-Change-Number: 15433
Gerrit-PatchSet: 4
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 28 Apr 2020 23:17:54 +0000
Gerrit-HasComments: Yes