Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Grant Henke, Hao Hao,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/13556
to look at the new patch set (#2).
Change subject: KUDU-2841. Expose HMS DB UUID to clients in HiveMetastoreConfig
......................................................................
KUDU-2841. Expose HMS DB UUID to clients in HiveMetastoreConfig
This fills in the hms_uuid field in HiveMetastoreConfig by fetching the
UUID from the HMS when the HMSCatalog starts up. The new API call is not
available in Hive 2.3 (in our thirdparty build) but has been backported
in some vendor distros and is available upstream in Hive 3.x. In order
to support it where available, this patch manually adds the call to the
thrift definition file, and catches the UNKNOWN_METHOD error code that
comes back when running against Hive 2.3.
I manually tested this against a Hive 3 master and verified that the
VLOG message at startup properly indicated the UUID of the HMS.
Change-Id: I393df4dd281c3fb63a836db346d7b33fb6edc6a2
---
M java/kudu-client/src/main/java/org/apache/kudu/client/HiveMetastoreConfig.java
M src/kudu/hms/hive_metastore.thrift
M src/kudu/hms/hms_catalog-test.cc
M src/kudu/hms/hms_catalog.cc
M src/kudu/hms/hms_catalog.h
M src/kudu/hms/hms_client.cc
M src/kudu/hms/hms_client.h
M src/kudu/master/master_service.cc
M src/kudu/thrift/client.cc
M src/kudu/thrift/client.h
10 files changed, 79 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/56/13556/2
--
To view, visit http://gerrit.cloudera.org:8080/13556
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I393df4dd281c3fb63a836db346d7b33fb6edc6a2
Gerrit-Change-Number: 13556
Gerrit-PatchSet: 2
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
