Hello Andrew Wong, Hao Hao, Todd Lipcon,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/13594
to review the following change.
Change subject: KUDU-2848: fetch HMS DB UUID in the background
......................................................................
KUDU-2848: fetch HMS DB UUID in the background
To avoid having a hard startup dependency from Kudu to HMS, let's thunk the
work of fetching the DB UUID to a helper thread. It's a simple approach, but
admittedly I haven't given much thought to whether it preserves reasonable
semantics for clients.
The new test isn't useful because our HMS test artifact doesn't support DB
UUIDs. I'll make sure this gets proper testing after merging though.
Change-Id: I5d190d317b3ee3e69493638e866949cef8fb5d46
---
M src/kudu/hms/hms_catalog-test.cc
M src/kudu/hms/hms_catalog.cc
M src/kudu/hms/hms_catalog.h
3 files changed, 84 insertions(+), 17 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/94/13594/1
--
To view, visit http://gerrit.cloudera.org:8080/13594
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5d190d317b3ee3e69493638e866949cef8fb5d46
Gerrit-Change-Number: 13594
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
