Sailesh Mukil has posted comments on this change.

Change subject: KUDU-1965: Allow user provided TLS certificates to work with 
KRPC
......................................................................


Patch Set 3:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/6555/1/src/kudu/security/cert.h
File src/kudu/security/cert.h:

PS1, Line 71: b
> nit: const ?
Done


PS1, Line 76: e:
> It seems this is not initialized -- consider adding an item into the list o
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ica6e2bacb378553723467f0dc54a166885db1e4d
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Sailesh Mukil <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Sailesh Mukil <[email protected]>
Gerrit-HasComments: Yes

Reply via email to