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>