[hive] branch master updated: HIVE-23213: HiveStrictManagedMigration should handle legacy Kudu tables (Adam Szita, reviewed by Peter Vary) addendum

[szita]

This is an automated email from the ASF dual-hosted git repository.

szita pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git


The following commit(s) were added to refs/heads/master by this push:
     new 2f7caa0  HIVE-23213: HiveStrictManagedMigration should handle legacy 
Kudu tables (Adam Szita, reviewed by Peter Vary) addendum
2f7caa0 is described below

commit 2f7caa0bf8882b481a9b2e12551bd7a0331bec6b
Author: Adam Szita <sz...@cloudera.com>
AuthorDate: Mon Apr 20 11:33:17 2020 +0200

    HIVE-23213: HiveStrictManagedMigration should handle legacy Kudu tables 
(Adam Szita, reviewed by Peter Vary) addendum
---
 .../java/org/apache/hadoop/hive/ql/util/HiveStrictManagedMigration.java | 2 ++
 1 file changed, 2 insertions(+)

diff --git 
a/ql/src/java/org/apache/hadoop/hive/ql/util/HiveStrictManagedMigration.java 
b/ql/src/java/org/apache/hadoop/hive/ql/util/HiveStrictManagedMigration.java
index 2c1ba5f..ab4cd6c 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/util/HiveStrictManagedMigration.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/util/HiveStrictManagedMigration.java
@@ -1036,6 +1036,8 @@ public class HiveStrictManagedMigration {
       }
       return true;
     case EXTERNAL_TABLE:
+      // Might need to update storage_handler
+      hiveUpdater.updateTableProperties(tableObj, new HashMap<>());
       msg = createExternalConversionExcuse(tableObj,
           "Table is already an external table");
       LOG.debug(msg);