[ 
https://issues.apache.org/jira/browse/HIVE-17771?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16204118#comment-16204118
 ] 

Sergey Shelukhin commented on HIVE-17771:
-----------------------------------------

The patch so far makes sense to me.
Nit: "   throw new SemanticException("Invalid syntax for CREATE RESOURCE PLAN 
statement is c/ped for show plan.
Also I see it passes null-s/false-s for permission stuff in HiveOperation.java. 
It seems to be in line with existing stuff where priviledges only apply to 
table/partition/view/etc. operations, but it was surprising to me.
[~thejas] I see that admin commands like grant role, etc. don't have any 
privileges associated with them in HiveOperation.java. How does one control 
access to that stuff?



> Implement create and show resource plan.
> ----------------------------------------
>
>                 Key: HIVE-17771
>                 URL: https://issues.apache.org/jira/browse/HIVE-17771
>             Project: Hive
>          Issue Type: Sub-task
>            Reporter: Harish Jaiprakash
>            Assignee: Harish Jaiprakash
>         Attachments: HIVE-17771.01.patch
>
>
> Please see parent jira about llap workload management.
> This jira is to implement create and show resource plan commands in hive to 
> configure resource plans for llap workload.
> The following are the proposed commands implemented as part of the jira:
> CREATE RESOURCE PLAN plan_name WITH QUERY_PARALLELISM parallelism;
> SHOW RESOURCE PLAN;
> It will be followed up with more jiras to add pools, triggers and copy 
> resource plans. And also with drop commands for each of them.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to