Fredy Wijaya has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/10030 )
Change subject: IMPALA-6451: AuthorizationException in CTAS for Kudu tables ...................................................................... IMPALA-6451: AuthorizationException in CTAS for Kudu tables CREATE TABLE on EXTERNAL Kudu tables or with TBLPROPERTIES ('kudu.master_addresses') require ALL privileges on SERVER. See IMPALA-4000. The patch fixes a bug where the CTAS statement is rewritten and during the initial analysis phase, 'kudu.master_addresses' property is added into TBLPROPERTIES which causes the next analysis phase to assume the statement was executed with TBLPROPERTIES ('kudu.master_addresses'). Hence, causing the ALL privilege request on SERVER to be registered. This patch also refactors the generated Kudu table name into a single place that stores all generated Kudu properties that gets cleared when the statement is reset. Testing: - Added a new test - Ran all front-end tests - Ran Kudu end-to-end query tests Change-Id: Iac3bbe4dceb80dfefd9756bc322f8c10ceab9f49 --- M fe/src/main/java/org/apache/impala/analysis/CreateTableStmt.java M fe/src/main/java/org/apache/impala/analysis/TableDef.java M fe/src/main/java/org/apache/impala/analysis/ToSqlUtils.java M fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java 4 files changed, 48 insertions(+), 19 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/30/10030/5 -- To view, visit http://gerrit.cloudera.org:8080/10030 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Iac3bbe4dceb80dfefd9756bc322f8c10ceab9f49 Gerrit-Change-Number: 10030 Gerrit-PatchSet: 5 Gerrit-Owner: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Adam Holley <g...@holleyism.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com>