Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11036 )
Change subject: hms tools: do not require HMS configuration flags ...................................................................... hms tools: do not require HMS configuration flags The HMS tools will now lookup required HMS configuration (hive_metastore_uris and hive_metastore_sasl_enabled) from the master when it is not provided via command line flags. Looking up the configs on the master is more ergonomic and less error-prone. Change-Id: Ibf580384a1612174a75142295d7f604cd6fabfe1 Reviewed-on: http://gerrit.cloudera.org:8080/11036 Reviewed-by: Hao Hao <hao....@cloudera.com> Tested-by: Kudu Jenkins --- M src/kudu/hms/hms_catalog.cc M src/kudu/mini-cluster/external_mini_cluster.cc M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/tool_action_common.cc M src/kudu/tools/tool_action_common.h M src/kudu/tools/tool_action_hms.cc 6 files changed, 157 insertions(+), 64 deletions(-) Approvals: Hao Hao: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/11036 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ibf580384a1612174a75142295d7f604cd6fabfe1 Gerrit-Change-Number: 11036 Gerrit-PatchSet: 15 Gerrit-Owner: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Kudu Jenkins