Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12653 )
Change subject: WIP [master] introduced SentryAuthzCache ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/12653/1/src/kudu/master/sentry_authz_cache.h File src/kudu/master/sentry_authz_cache.h: http://gerrit.cloudera.org:8080/#/c/12653/1/src/kudu/master/sentry_authz_cache.h@50 PS1, Line 50: SentryAuthzCache > The API this exposes isn't very cache-like (e.g. no Put), so it probably de These are good suggestions. I'll try to incorporate those into the alternative patch that uses newly introduced FIFO cache from util/cache.{cc,h} -- To view, visit http://gerrit.cloudera.org:8080/12653 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If377cd4a7bae481ffc9a6950cb26862b66e9bab5 Gerrit-Change-Number: 12653 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Mon, 18 Mar 2019 18:11:02 +0000 Gerrit-HasComments: Yes
