You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Dan Burkert (Code Review)" <ge...@cloudera.org> on 2018/04/02 19:00:14 UTC

[kudu-CR] KUDU-2191 (7/n): HmsCatalog

Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, Todd Lipcon, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/9862

to look at the new patch set (#3).

Change subject: KUDU-2191 (7/n): HmsCatalog
......................................................................

KUDU-2191 (7/n): HmsCatalog

This commit adds some important higher-level HMS machinery in the
HmsCatalog class.  This class is responsible for translating Kudu
catalog information to the Hive format, thread safety, retries, Hive HA,
and configuration flags for the Hive integration.

A follow-up commit will integrate the HmsCatalog into the
CatalogManager, at which point many parts of the HMS integration will
become functional. Until that point this is still non-functional code.

Change-Id: I6eb2a4c400f5aaee095e4e4ad572981565a1c040
---
M src/kudu/hms/CMakeLists.txt
A src/kudu/hms/hms_catalog-test.cc
A src/kudu/hms/hms_catalog.cc
A src/kudu/hms/hms_catalog.h
M src/kudu/hms/mini_hms.cc
M src/kudu/hms/mini_hms.h
M src/kudu/master/CMakeLists.txt
M src/kudu/master/master.cc
8 files changed, 1,108 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/62/9862/3
-- 
To view, visit http://gerrit.cloudera.org:8080/9862
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I6eb2a4c400f5aaee095e4e4ad572981565a1c040
Gerrit-Change-Number: 9862
Gerrit-PatchSet: 3
Gerrit-Owner: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>