Bharath Vissapragada has uploaded a new change for review. http://gerrit.cloudera.org:8080/2664
Change subject: IMPALA-3269: Remove authz checks on default table location in CTAS queries ...................................................................... IMPALA-3269: Remove authz checks on default table location in CTAS queries Bug: In CreateTableAsSelectStmt.analyze(), we set the default location of table if the query doesn't explicitly set a table location. However this is an issue with CTAS with subqueries as they follow a two pass analysis with the second analyze() call failing with a authz error on the URI added in the first pass. Fix: We needn't set the default location explicitly. Metastore automatically figures it out if it is not set. Change-Id: I0451586d4994cf9fc8c3dd47c8f3a513067cb2ea --- M fe/src/main/java/com/cloudera/impala/analysis/CreateTableAsSelectStmt.java M fe/src/main/java/com/cloudera/impala/analysis/CreateTableLikeFileStmt.java M fe/src/main/java/com/cloudera/impala/analysis/ToSqlUtils.java M fe/src/test/java/com/cloudera/impala/analysis/AuthorizationTest.java M fe/src/test/java/com/cloudera/impala/analysis/ToSqlTest.java 5 files changed, 12 insertions(+), 16 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/64/2664/1 -- To view, visit http://gerrit.cloudera.org:8080/2664 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I0451586d4994cf9fc8c3dd47c8f3a513067cb2ea Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Bharath Vissapragada <[email protected]>
