Dan Burkert has posted comments on this change.

Change subject: [security]  security-flags
......................................................................


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/6052/2//COMMIT_MSG
Commit Message:

PS2, Line 43:  2) It's really long, and the length doesn't add
            :     useful description or specificity. 3) The short form (cert 
instead
            :     of certificate) is common in database CLI configs [1], [2], 
[3].
> yea, I agree that 'rpc_key' isn't very descriptive. At the least I'd expect
Done


Line 58: --webserver_certificate_file
> Sure, I also don't really care too much about the RPC settings for now. Sam
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa53348b8969e83d9f794e1e0553bdec12252d9a
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to