Todd Lipcon has posted comments on this change.

Change subject: [security] added info on non-renewable authn tokens
......................................................................


Patch Set 1:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/6122/1/src/kudu/master/master.cc
File src/kudu/master/master.cc:

PS1, Line 64: re-new
nit: "renew" one word


PS1, Line 66: token
"a token"


PS1, Line 67: // TODO(PKI): docs for what actual effect this has, given we 
don't support
            : // token renewal.
            : T
remove this TODO?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I44b5aedb05803ffba7a22f8127b2edac60d3752c
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to