Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Greg Solovyev, Hao Hao,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14179
to look at the new patch set (#3).
Change subject: [hms] Sync external HMS tables with purge property
......................................................................
[hms] Sync external HMS tables with purge property
This patch is another patch in preparation for Hive 3 and 4.
In Hive 3 the meaning of managed tables is a bit more strict.
To replace the old behavior of a managed table, external tables
now have the `external.table.purge` property. Further, in
Hive 4, non-transactional Hive tables will get auto-migrated
to external tables with `external.table.purge=true`.
With this change, the HMS sync now treats managed tables and
external tables with `external.table.purge=true` as synchronized tables.
External tables with `external.table.purge=false` will continue to be
ingnored like all external tables before.
To avoid orphaned tables, the HMS plugin was changed to restrict changing
the `external.table.purge` property. This is similar to the existing
functionality which prevented changing the table type.
Last, the HMS tooling was updated to differentiate between synchronized
and unsynchronized tables instead of explicitly managed and external tables.
Change-Id: I942515c015b1a4a63cf08383c331c4e19350c312
---
M
java/kudu-hive/src/main/java/org/apache/kudu/hive/metastore/KuduMetastorePlugin.java
M
java/kudu-hive/src/test/java/org/apache/kudu/hive/metastore/TestKuduMetastorePlugin.java
M src/kudu/hms/hms_catalog.cc
M src/kudu/hms/hms_client.cc
M src/kudu/hms/hms_client.h
M src/kudu/integration-tests/hms_itest-base.cc
M src/kudu/integration-tests/hms_itest-base.h
M src/kudu/integration-tests/master_hms-itest.cc
M src/kudu/master/hms_notification_log_listener.cc
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/tool_action_hms.cc
11 files changed, 237 insertions(+), 58 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/79/14179/3
--
To view, visit http://gerrit.cloudera.org:8080/14179
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I942515c015b1a4a63cf08383c331c4e19350c312
Gerrit-Change-Number: 14179
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Greg Solovyev <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
