Alexey Serbin 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 Done PS1, Line 66: token > "a token" Done PS1, Line 67: // TODO(PKI): docs for what actual effect this has, given we don't support : // token renewal. : T > remove this TODO? Yup, since the flag's command line doc has been updated and KUDU-1895 is filed, this is no longer need. Thanks. -- 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: 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
