Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13594 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/13594 Tested-by: Kudu Jenkins Reviewed-by: Grant Henke <[email protected]> --- 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(-) Approvals: Kudu Jenkins: Verified Grant Henke: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I5d190d317b3ee3e69493638e866949cef8fb5d46 Gerrit-Change-Number: 13594 Gerrit-PatchSet: 2 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Todd Lipcon <[email protected]>
