Bankim Bhavsar has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15889 )

Change subject: Suggesting a secure implementation logic of the X509TrustManager
......................................................................


Patch Set 1:

Indentation looks way off.
Kudu uses 2 spaces for indentation for C++ and Java code. 
https://kudu.apache.org/docs/contributing.html#_c_code_style
https://google.github.io/styleguide/cppguide.html#Spaces_vs._Tabs


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ice36ca72a4e31b381033be68a40925bfb729bbe6
Gerrit-Change-Number: 15889
Gerrit-PatchSet: 1
Gerrit-Owner: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 08 May 2020 22:58:07 +0000
Gerrit-HasComments: No

Reply via email to