Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/17445 )

Change subject: [hms] Allow the table type to change but not the 
synchronization of a table
......................................................................

[hms] Allow the table type to change but not the synchronization of a table

When upgrading some verions of the Hive Metastore it migrates Kudu tables
from managed tables to external tables with the purge property set. Both
of these states are considered “synchronized” tables by the HMS sync
and therefore this change sholuld be allowed as it won’t created orphaned
tables.

Change-Id: I7c029da1dbfef9812b0fe7389a768772605acc45
Reviewed-on: http://gerrit.cloudera.org:8080/17445
Reviewed-by: Attila Bukor <[email protected]>
Tested-by: Grant Henke <[email protected]>
---
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
2 files changed, 34 insertions(+), 22 deletions(-)

Approvals:
  Attila Bukor: Looks good to me, approved
  Grant Henke: Verified

--
To view, visit http://gerrit.cloudera.org:8080/17445
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I7c029da1dbfef9812b0fe7389a768772605acc45
Gerrit-Change-Number: 17445
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to