Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9863 )
Change subject: KUDU-2191 (8/n): Integrate HmsCatalog into CatalogManager ...................................................................... KUDU-2191 (8/n): Integrate HmsCatalog into CatalogManager This commit connects the CatalogManager to the HMS via the HmsCatalog abstraction. When the HMS integration is enabled (by setting the --hive-metastore-uris flag) and Kudu tables are created, altered, or dropped, the corresponding HMS entry is also modified appropriately. Additionally, New table and column names are required to be valid according to the Hive identifier rules, which are much stricter than Kudu's existing identifier rules. Testing: This commit adds a new integration test (master_hms-itest) which tests that the integration works as expected with create/alter/drop table operations. Additionally, some existing DDL stress tests now have the HMS integration enabled in order to provide more coverage. Change-Id: Ie68e143c3c317c7690af097e6485934feb1010b4 Reviewed-on: http://gerrit.cloudera.org:8080/9863 Reviewed-by: Dan Burkert <[email protected]> Tested-by: Dan Burkert <[email protected]> --- M build-support/iwyu/iwyu-filter.awk M src/kudu/hms/hms_catalog.cc M src/kudu/hms/hms_catalog.h M src/kudu/hms/hms_client.h M src/kudu/hms/mini_hms.h M src/kudu/integration-tests/CMakeLists.txt M src/kudu/integration-tests/alter_table-randomized-test.cc M src/kudu/integration-tests/master-stress-test.cc M src/kudu/integration-tests/master_failover-itest.cc A src/kudu/integration-tests/master_hms-itest.cc M src/kudu/master/CMakeLists.txt M src/kudu/master/catalog_manager.cc M src/kudu/master/catalog_manager.h M src/kudu/master/master.proto M src/kudu/mini-cluster/external_mini_cluster.cc 15 files changed, 655 insertions(+), 89 deletions(-) Approvals: Dan Burkert: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/9863 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ie68e143c3c317c7690af097e6485934feb1010b4 Gerrit-Change-Number: 9863 Gerrit-PatchSet: 9 Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]>
