Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Hao Hao,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/12833
to look at the new patch set (#2).
Change subject: WIP [master] introduced SentryAuthzCache
......................................................................
WIP [master] introduced SentryAuthzCache
This patch incorporate a TTL-based cache into the SentryAuthzProvider.
The cache stores responses received from the authz provider (Sentry in
this case). It's possible to enable or disable using the cache upon
creation of SentryAuthzProvider instance: set the newly introduced
`--sentry_cache_capacity` command-line flag to 0 to disable the cache.
WIP:
* update current tests to account for the cache presence
(e.g., parameterize them)
* to add cache-specific tests
* maybe, pre-process raw TListSentryPrivilegesResponse into some sort
of structure to speed up lookups of certain privilege for
authorizables
Change-Id: Idaefacd50736f1f152dae34e76778e17b2e84cbe
---
M src/kudu/master/CMakeLists.txt
A src/kudu/master/sentry_authz_cache.cc
A src/kudu/master/sentry_authz_cache.h
M src/kudu/master/sentry_authz_provider-test.cc
M src/kudu/master/sentry_authz_provider.cc
M src/kudu/master/sentry_authz_provider.h
6 files changed, 354 insertions(+), 11 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/33/12833/2
--
To view, visit http://gerrit.cloudera.org:8080/12833
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Idaefacd50736f1f152dae34e76778e17b2e84cbe
Gerrit-Change-Number: 12833
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
