Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9071 )

Change subject: [java] add test to reproduce KUDU-2267
......................................................................


Patch Set 1:

(4 comments)

What are we going to do with this test once the server-side issue with not 
having CA-signed certs for follower masters is fixed?

http://gerrit.cloudera.org:8080/#/c/9071/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/9071/1//COMMIT_MSG@11
PS1, Line 11: credential
nit: in English it usually goes plural, i.e. 'credentials'


http://gerrit.cloudera.org:8080/#/c/9071/1//COMMIT_MSG@13
PS1, Line 13:  that masters
nit: drop


http://gerrit.cloudera.org:8080/#/c/9071/1//COMMIT_MSG@13
PS1, Line 13: add
nit: adds


http://gerrit.cloudera.org:8080/#/c/9071/1//COMMIT_MSG@14
PS1, Line 14:  in TestSecurity
nit: drop



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I4879749988dc884fe81cf36838819379db91ae72
Gerrit-Change-Number: 9071
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Comment-Date: Sat, 20 Jan 2018 02:39:00 +0000
Gerrit-HasComments: Yes

Reply via email to