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

Harish Jaiprakash edited comment on HIVE-18134 at 11/23/17 5:26 AM:
--------------------------------------------------------------------

Thanks Sergey. I've to make similar changes for stripping quotes at other 
places like policy.

Just one comment, Should we use identifier to restrict to top level or poolPath 
to enable any path to be parsed, instead of String?

I've changed alter resource plan in HIVE-17954, to check for disabled state for 
not status changes at the API end.


was (Author: harishjp):
Thanks Sergey. I've to make similar changes for stripping quotes at other 
places like policy.

Just one comment, Should we use identifier to restrict to top level or poolPath 
to enable any path to be parsed, instead of String.

I've changed alter resource plan in HIVE-17954, to check for disabled state for 
not status changes and made it bit more intuitive.

> some alter resource plan fixes
> ------------------------------
>
>                 Key: HIVE-18134
>                 URL: https://issues.apache.org/jira/browse/HIVE-18134
>             Project: Hive
>          Issue Type: Sub-task
>            Reporter: Sergey Shelukhin
>            Assignee: Sergey Shelukhin
>         Attachments: HIVE-18134.patch
>
>
> Part of HIVE-18075



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

Reply via email to