Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13227
Change subject: [authz] validator for --sentry_service_rpc_addresses flag ...................................................................... [authz] validator for --sentry_service_rpc_addresses flag This patch introduces a group validator for the --sentry_service_rpc_addresses runtime flag. It makes it's necessary to set a non-empty value for the --hive_metastore_uris flag if the --sentry_service_rpc_addresses flag is set. In other words, this patch makes it explicitly impossible to run Kudu master for a hypothetical configuration of Kudu+Sentry authz without HMS catalog. That reflects the logical dependency of the Kudu+Sentry fine-grain authz scheme on the HMS catalog. The dependency is there by design. This patchs also contains a test for the introduced flag validator. As of now, all existing tests for Kudu+Sentry authz are run with configuration where the integration with HMS catalog is enabled as well. Change-Id: Iec0470f68e34edf72a9e8baf608eda1b83272921 --- M src/kudu/integration-tests/master_sentry-itest.cc M src/kudu/master/sentry_authz_provider-test.cc M src/kudu/master/sentry_authz_provider.cc M src/kudu/master/sentry_privileges_fetcher.cc 4 files changed, 44 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/27/13227/1 -- To view, visit http://gerrit.cloudera.org:8080/13227 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Iec0470f68e34edf72a9e8baf608eda1b83272921 Gerrit-Change-Number: 13227 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
