Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13318
Change subject: IMPALA-8504: Support CREATE TABLE statement with Kudu/HMS integration ...................................................................... IMPALA-8504: Support CREATE TABLE statement with Kudu/HMS integration This commits support the syntax of CREATE TABLE (and CATS) statments for both managed and external Kudu tables with Kudu/HMS integration. Syntax example for external table with Kudu/HMS integration enabled: CREATE EXTERNAL TABLE t STORED AS KUDU TBLPROPERTIES For managed table the syntax remains the same. The detailed changes includes: 1) Add the new Kudu storage handler 'org.apache.kudu.hive.KuduStorageHandler' that is used in Kudu/HMS integration. Tables with either the legacy storage handler 'com.cloudera.kudu.hive.KuduStorageHandler' and the one are considered as Kudu tables. 2) When Kudu/HMS integration is enabled, the external table is no longer allowed to set table property TABLE_NAME. The name of underlying Kudu table is derived from the HMS table with the format 'db_name.table_name'. 3) Add 'kudu.table_id' table property for both managed and external table to match with the new metadata format with Kudu/HMS integration. This commits also extracts Kudu related DDL parsing and analyzing tests, so that they can be run with or without Kudu/HMS integration enabled. Change-Id: I465673d749221bd5f3772814b1c22c2673a53f5c --- M fe/src/main/java/org/apache/impala/analysis/CreateTableStmt.java M fe/src/main/java/org/apache/impala/analysis/ToSqlUtils.java M fe/src/main/java/org/apache/impala/catalog/KuduTable.java M fe/src/main/java/org/apache/impala/catalog/local/LocalKuduTable.java M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java M fe/src/main/java/org/apache/impala/util/KuduUtil.java M fe/src/test/java/org/apache/impala/analysis/AnalyzeDDLTest.java A fe/src/test/java/org/apache/impala/analysis/AnalyzeKuduDDLTest.java A fe/src/test/java/org/apache/impala/analysis/AuditingKuduTest.java M fe/src/test/java/org/apache/impala/analysis/AuditingTest.java A fe/src/test/java/org/apache/impala/analysis/KuduHMSIntegrationTest.java M fe/src/test/java/org/apache/impala/analysis/ToSqlTest.java M fe/src/test/java/org/apache/impala/common/FrontendTestBase.java 13 files changed, 884 insertions(+), 595 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/18/13318/1 -- To view, visit http://gerrit.cloudera.org:8080/13318 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I465673d749221bd5f3772814b1c22c2673a53f5c Gerrit-Change-Number: 13318 Gerrit-PatchSet: 1 Gerrit-Owner: Hao Hao <[email protected]>
