Vihang Karajgaonkar has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/14397


Change subject: IMPALA-9030: Handle translated external Kudu tables
......................................................................

IMPALA-9030: Handle translated external Kudu tables

In HMS 3.1 there is a default tranformer introduced which checks the client 
capabilities
and transforms a table before creating it. Additionally, it also makes sure 
that any
managed table which is created is transactional. If a user creates a managed 
table which
is not transactional, it automatically converts such table as external and sets 
certain
table properties to mark such transformed tables.

This presents a problem for managed Kudu tables in Impala since managed and 
external
tables are handled differently in Kudu. Specifically, if a Kudu table is 
managed, certain
operations like drop table, rename table, alter table are performed on the Kudu 
side along
with updating the catalog. If the Kudu table is external, the Kudu operations 
are skipped
and only catalog side operations are performed.

When the user creates a managed Kudu table, user expects that drop table, 
rename table
should be updated by Impala automatically in Kudu as well. But since HMS 3 
transforms such
managed tables into external, currently Impala does not perform the Kudu side 
operations
breaking the semantics for the user.

This patch makes changes to Catalog so that it can detect such transformed 
external tables
and perform Kudu side operations similar to what it was doing for managed Kudu 
table when
talking with previous HMS versions.

Testing:
1. Ran existing Kudu tests which were failing when the CDP build number was 
bumped up.
2. Running all tests with USE_CDP_HIVE flag set to true.

Change-Id: I324523361c923b7d291cb4d0f1028b1a5b653b36
---
M fe/src/main/java/org/apache/impala/analysis/AlterTableSetTblProperties.java
M fe/src/main/java/org/apache/impala/catalog/KuduTable.java
M fe/src/main/java/org/apache/impala/catalog/Table.java
M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java
M fe/src/main/java/org/apache/impala/service/KuduCatalogOpExecutor.java
M fe/src/test/java/org/apache/impala/common/FrontendFixture.java
6 files changed, 38 insertions(+), 25 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/97/14397/1
--
To view, visit http://gerrit.cloudera.org:8080/14397
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I324523361c923b7d291cb4d0f1028b1a5b653b36
Gerrit-Change-Number: 14397
Gerrit-PatchSet: 1
Gerrit-Owner: Vihang Karajgaonkar <[email protected]>

Reply via email to