Repository: impala Updated Branches: refs/heads/2.x 3acb8f984 -> c66096523
IMPALA-6887: Typo in authz-policy.ini.template Before: alter_functionl_text_lzo After: alter_functional_text_lzo This patch also adds missing test cases for ALTER privilege on functional_text_lzo database. Testing: - Ran all front-end tests Change-Id: I6aea8d71dda39838e9e70160018ce2c5fc73df21 Reviewed-on: http://gerrit.cloudera.org:8080/10113 Reviewed-by: Bharath Vissapragada <bhara...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Project: http://git-wip-us.apache.org/repos/asf/impala/repo Commit: http://git-wip-us.apache.org/repos/asf/impala/commit/c6609652 Tree: http://git-wip-us.apache.org/repos/asf/impala/tree/c6609652 Diff: http://git-wip-us.apache.org/repos/asf/impala/diff/c6609652 Branch: refs/heads/2.x Commit: c66096523d455901055b040c6c17759d08b06bc0 Parents: c226f0e Author: Fredy wijaya <fwij...@cloudera.com> Authored: Wed Apr 18 18:19:23 2018 -0700 Committer: Fredy Wijaya <fwij...@cloudera.com> Committed: Wed Apr 25 13:22:15 2018 -0700 ---------------------------------------------------------------------- .../impala/analysis/AuthorizationTest.java | 20 ++++++++++++++++++++ fe/src/test/resources/authz-policy.ini.template | 2 +- 2 files changed, 21 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/impala/blob/c6609652/fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java ---------------------------------------------------------------------- diff --git a/fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java b/fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java index c82b4c8..e5b7702 100644 --- a/fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java +++ b/fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java @@ -1472,6 +1472,26 @@ public class AuthorizationTest extends FrontendTestBase { AuthzOk("ALTER TABLE functional.alltypeserror SET CACHED IN 'testPool'"); AuthzOk("ALTER TABLE functional.alltypeserror RECOVER PARTITIONS"); + // User has ALTER privilege on functional_text_lzo database. + AuthzOk("ALTER TABLE functional_text_lzo.alltypeserror ADD COLUMNS (c1 int)"); + AuthzOk("ALTER TABLE functional_text_lzo.alltypeserror REPLACE COLUMNS (c1 int)"); + AuthzOk("ALTER TABLE functional_text_lzo.alltypeserror CHANGE id c1 int"); + AuthzOk("ALTER TABLE functional_text_lzo.alltypeserror DROP id"); + AuthzOk("ALTER TABLE functional_text_lzo.alltypeserror RENAME TO " + + "functional_seq_snap.t1"); + AuthzOk("ALTER TABLE functional_text_lzo.alltypeserror SET FILEFORMAT PARQUET"); + AuthzOk("ALTER TABLE functional_text_lzo.alltypeserror SET LOCATION " + + "'/test-warehouse/new_table'"); + AuthzOk("ALTER TABLE functional_text_lzo.alltypeserror SET TBLPROPERTIES " + + "('a'='b', 'c'='d')"); + AuthzOk("ALTER TABLE functional_text_lzo.alltypeserror SET LOCATION " + + "'hdfs://localhost:20500/test-warehouse/new_table'"); + AuthzOk("ALTER TABLE functional_text_lzo.alltypeserror " + + "PARTITION(year=2009, month=1) SET LOCATION " + + "'hdfs://localhost:20500/test-warehouse/new_table'"); + AuthzOk("ALTER TABLE functional_text_lzo.alltypeserror SET CACHED IN 'testPool'"); + AuthzOk("ALTER TABLE functional_text_lzo.alltypeserror RECOVER PARTITIONS"); + // Alter table and set location to a path the user does not have access to. // User needs ALTER on table and ALL on URI. AuthzError("ALTER TABLE functional_seq_snap.alltypes SET LOCATION " + http://git-wip-us.apache.org/repos/asf/impala/blob/c6609652/fe/src/test/resources/authz-policy.ini.template ---------------------------------------------------------------------- diff --git a/fe/src/test/resources/authz-policy.ini.template b/fe/src/test/resources/authz-policy.ini.template index 26ed70b..e26ee55 100644 --- a/fe/src/test/resources/authz-policy.ini.template +++ b/fe/src/test/resources/authz-policy.ini.template @@ -68,7 +68,7 @@ alter_functional_alltypes_view =\ server=server1->db=functional->table=alltypes_view->action=alter insert_functional_text_lzo = server=server1->db=functional_text_lzo->action=insert create_functional_text_lzo = server=server1->db=functional_text_lzo->action=create -alter_functionl_text_lzo = server=server1->db=functional_text_lzo->action=alter +alter_functional_text_lzo = server=server1->db=functional_text_lzo->action=alter drop_functional_text_lzo = server=server1->db=functional_text_lzo->action=drop select_column_level_functional =\ server=server1->db=functional->table=alltypessmall->column=id->action=select,\