Hello Andrew Wong, Adar Dembo, Hao Hao,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/13139

to look at the new patch set (#5).

Change subject: [tools] command to reset authz cache in kudu CLI
......................................................................

[tools] command to reset authz cache in kudu CLI

Added command to reset authz privileges cache into kudu CLI.
Synopsis:

  kudu master authz_cache reset <all_master_addresses_in_cluster>

Added corresponding tests as well.  A scenario for a 'positive' case
is not among the added ones: it will be a separate effort to bring in
HMS+Sentry into the framework for testing kudu CLI tool.  However,
there is coverage for a 'positive' case for cache reset elsewhere.
In particular, the scenarios of the SentryAuthzProviderCacheITest
give appropriate coverage using the ResetAuthzCache RPC endpoint.
Those scenarios test and AdminCliTest.TestAuthzResetCacheNotImplemented
should provide enough coverage.

Change-Id: If49f3ebdea22ea0df3aaec313b4db949dc834943
---
M src/kudu/tools/kudu-admin-test.cc
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/tool_action_master.cc
3 files changed, 255 insertions(+), 69 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/39/13139/5
--
To view, visit http://gerrit.cloudera.org:8080/13139
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If49f3ebdea22ea0df3aaec313b4db949dc834943
Gerrit-Change-Number: 13139
Gerrit-PatchSet: 5
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>

Reply via email to