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

Thejas M Nair commented on HIVE-8685:
-------------------------------------

How about going to back to the original goal of SecureProxySupport - " it's a 
noop if security is not enabled." (as mentioned in the description).

Ie, moving args.add("proxy.user.name=" + user); back under if(isEnabled)

Also, can you run the code you have added/modified through the IDE formatter ? 
There are a few nits like " <=0" which when properly formatted would be " <= 0" 
(space after operator).


> DDL operations in WebHCat set proxy user to "null" in unsecure mode
> -------------------------------------------------------------------
>
>                 Key: HIVE-8685
>                 URL: https://issues.apache.org/jira/browse/HIVE-8685
>             Project: Hive
>          Issue Type: Bug
>          Components: WebHCat
>    Affects Versions: 0.14.0
>            Reporter: Eugene Koifman
>            Assignee: Eugene Koifman
>            Priority: Critical
>         Attachments: HIVE-8685.patch
>
>
> This makes DDL commands fail
> This was stupidly broken in HIVE-8643
> NO PRECOMMIT TESTS



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to