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 (#4).

Change subject: WIP [master] introduced SentryAuthzCache
......................................................................

WIP [master] introduced SentryAuthzCache

This patch incorporates a TTL-based cache into the SentryAuthzProvider.
The cache stores responses received from the authz provider, i.e. Sentry
in this particular case.  It's possible to enable or disable using
the caching of Sentry responses upon creation of SentryAuthzProvider
instance: set the newly introduced `--sentry_authz_cache_capacity_bytes`
command-line flag to 0 to disable the cache.

In addition, it's possible to force the cache to fetch and cache
information from higher level of Sentry's authz hierarchy: use the
`--sentry_authz_cache_finest_scope` flag for that.

WIP:
  * to add tests specific to SentryAuthzCache:
      ** for various --sentry_authz_cache_finest_scope

Change-Id: Idaefacd50736f1f152dae34e76778e17b2e84cbe
---
M src/kudu/master/CMakeLists.txt
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
4 files changed, 394 insertions(+), 73 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/33/12833/4
--
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: 4
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)

Reply via email to