Alexey Serbin has posted comments on this change. Change subject: [security] load/store public TSK in the system table ......................................................................
Patch Set 9: > (34 comments) > > I did a first pass. I think the most important question to settle > is whether followers need to maintain an up-to-date TSK soft state. > I'd prefer the answer to be "no", as it'd simplify this code a lot. Thank you for the review. Per our Slack discussion on kudu-security we decided that followers do not maintain up-to-date TSK soft state (i.e. no system table polling from follower masters). That and the decision to store private part of the key in the table as well simplified the code, indeed. -- To view, visit http://gerrit.cloudera.org:8080/5935 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie91d4129bda0ca49e81988c28385895a2abcd201 Gerrit-PatchSet: 9 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <[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: Tidy Bot Gerrit-HasComments: No
