You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by se...@apache.org on 2017/10/25 18:33:44 UTC

hive git commit: HIVE-17771 : Implement commands to manage resource plan (Harish Jaiprakash, reviewed by Sergey Shelukhin) ADDENDUM2

Repository: hive
Updated Branches:
  refs/heads/master eb92b4fc4 -> 75d1bf350


HIVE-17771 : Implement commands to manage resource plan (Harish Jaiprakash, reviewed by Sergey Shelukhin) ADDENDUM2


Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/75d1bf35
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/75d1bf35
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/75d1bf35

Branch: refs/heads/master
Commit: 75d1bf35070b2a00ec08c511534b651b08ec7371
Parents: eb92b4f
Author: sergey <se...@apache.org>
Authored: Wed Oct 25 11:33:20 2017 -0700
Committer: sergey <se...@apache.org>
Committed: Wed Oct 25 11:33:20 2017 -0700

----------------------------------------------------------------------
 .../plugin/sqlstd/Operation2Privilege.java            | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/75d1bf35/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/Operation2Privilege.java
----------------------------------------------------------------------
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/Operation2Privilege.java b/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/Operation2Privilege.java
index 366737a..14dedde 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/Operation2Privilege.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/Operation2Privilege.java
@@ -311,6 +311,10 @@ public class Operation2Privilege {
     adminPrivOps.add(HiveOperationType.REPLDUMP);
     adminPrivOps.add(HiveOperationType.REPLLOAD);
     adminPrivOps.add(HiveOperationType.KILL_QUERY);
+    adminPrivOps.add(HiveOperationType.CREATE_RESOURCEPLAN);
+    adminPrivOps.add(HiveOperationType.ALTER_RESOURCEPLAN);
+    adminPrivOps.add(HiveOperationType.DROP_RESOURCEPLAN);
+    adminPrivOps.add(HiveOperationType.SHOW_RESOURCEPLAN);
 
     // operations require select priv
     op2Priv.put(HiveOperationType.SHOWCOLUMNS, PrivRequirement.newIOPrivRequirement
@@ -432,8 +436,8 @@ public class Operation2Privilege {
         null));
     op2Priv.put(HiveOperationType.SHOW_GRANT, PrivRequirement.newIOPrivRequirement
 (null, null));
-    op2Priv.put(HiveOperationType.GRANT_ROLE, PrivRequirement.newIOPrivRequirement
-(null, null));
+      op2Priv.put(HiveOperationType.GRANT_ROLE, PrivRequirement.newIOPrivRequirement
+  (null, null));
     op2Priv.put(HiveOperationType.REVOKE_ROLE, PrivRequirement.newIOPrivRequirement
 (null, null));
     op2Priv.put(HiveOperationType.SHOW_ROLES, PrivRequirement.newIOPrivRequirement
@@ -466,7 +470,13 @@ public class Operation2Privilege {
             arr(SQLPrivTypeGrant.SELECT_NOGRANT, SQLPrivTypeGrant.DELETE_NOGRANT), INS_NOGRANT_AR));
     op2Priv.put(HiveOperationType.ABORT_TRANSACTIONS, PrivRequirement.newIOPrivRequirement
       (null, null));
+
+    // Handled via adminPrivOps (see above).
     op2Priv.put(HiveOperationType.KILL_QUERY, PrivRequirement.newIOPrivRequirement(null, null));
+    op2Priv.put(HiveOperationType.CREATE_RESOURCEPLAN, PrivRequirement.newIOPrivRequirement(null, null));
+    op2Priv.put(HiveOperationType.ALTER_RESOURCEPLAN, PrivRequirement.newIOPrivRequirement(null, null));
+    op2Priv.put(HiveOperationType.DROP_RESOURCEPLAN, PrivRequirement.newIOPrivRequirement(null, null));
+    op2Priv.put(HiveOperationType.SHOW_RESOURCEPLAN, PrivRequirement.newIOPrivRequirement(null, null));
   }
 
   /**