Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9388 )
Change subject: [java] fix the regression test for KUDU-2267/KUDU-2319 ...................................................................... Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/9388/1/java/kudu-client/src/main/java/org/apache/kudu/client/ConnectToCluster.java File java/kudu-client/src/main/java/org/apache/kudu/client/ConnectToCluster.java: http://gerrit.cloudera.org:8080/#/c/9388/1/java/kudu-client/src/main/java/org/apache/kudu/client/ConnectToCluster.java@163 PS1, Line 163: KuduTable masterTable, : KuduRpc<?> parentRpc, : long defaultTimeoutMs, > nit: padding Done http://gerrit.cloudera.org:8080/#/c/9388/1/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/9388/1/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java@83 PS1, Line 83: > A Done http://gerrit.cloudera.org:8080/#/c/9388/1/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java@87 PS1, Line 87: */ > If you base this a top of https://gerrit.cloudera.org/c/9373/, you can enab Right, I checked out the fix and then this test is passing -- To view, visit http://gerrit.cloudera.org:8080/9388 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I5e827586fe549f6a0c927ce8a4f8eca954bfe690 Gerrit-Change-Number: 9388 Gerrit-PatchSet: 2 Gerrit-Owner: Hao Hao <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Fri, 23 Feb 2018 07:38:51 +0000 Gerrit-HasComments: Yes
