You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by ta...@apache.org on 2018/04/19 18:12:18 UTC
[7/8] impala git commit: IMPALA-6887: Typo in
authz-policy.ini.template
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 <bh...@cloudera.com>
Tested-by: Impala Public Jenkins <im...@cloudera.com>
Project: http://git-wip-us.apache.org/repos/asf/impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/impala/commit/08a1a7a9
Tree: http://git-wip-us.apache.org/repos/asf/impala/tree/08a1a7a9
Diff: http://git-wip-us.apache.org/repos/asf/impala/diff/08a1a7a9
Branch: refs/heads/master
Commit: 08a1a7a967f19e41e8523a9020320bbad5cdac77
Parents: 5d328f4
Author: Fredy wijaya <fw...@cloudera.com>
Authored: Wed Apr 18 18:19:23 2018 -0700
Committer: Impala Public Jenkins <im...@cloudera.com>
Committed: Thu Apr 19 07:23:29 2018 +0000
----------------------------------------------------------------------
.../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/08a1a7a9/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 f0b7332..b04e138 100644
--- a/fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java
+++ b/fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java
@@ -1479,6 +1479,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/08a1a7a9/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,\