You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Alex Behm (Code Review)" <ge...@cloudera.org> on 2016/04/06 08:22:36 UTC

[Impala-CR](cdh5-trunk) IMPALA-3269: Remove authz checks on default table location in CTAS queries

Alex Behm has posted comments on this change.

Change subject: IMPALA-3269: Remove authz checks on default table location in CTAS queries
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/2664/1/fe/src/test/java/com/cloudera/impala/analysis/AuthorizationTest.java
File fe/src/test/java/com/cloudera/impala/analysis/AuthorizationTest.java:

Line 826:     AuthzOk("create table tpch.new_table as select * from functional.alltypesagg " +
Why did this test fail before your patch? Just want to make sure I understand.


-- 
To view, visit http://gerrit.cloudera.org:8080/2664
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I0451586d4994cf9fc8c3dd47c8f3a513067cb2ea
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Bharath Vissapragada <bh...@cloudera.com>
Gerrit-Reviewer: Alex Behm <al...@cloudera.com>
Gerrit-HasComments: Yes