[jira] [Comment Edited] (FALCON-595) Improvements to DefaultAuthorizationProvider code

["Venkatesh Seetharam (JIRA)" <ji...@apache.org>]

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

Venkatesh Seetharam edited comment on FALCON-595 at 9/2/14 11:59 PM:
---------------------------------------------------------------------

Looks good. Why is this removed?
{code}
-        HashSet<String> s = new HashSet<String>(Arrays.asList(proxyUgi.getGroupNames()));
-        return Collections.unmodifiableSet(s);
{code}

Did you verify the tests pass?


was (Author: svenkat):
Looks good. Why is this removed?
{code}
-        HashSet<String> s = new HashSet<String>(Arrays.asList(proxyUgi.getGroupNames()));
-        return Collections.unmodifiableSet(s);
{code}

> Improvements to DefaultAuthorizationProvider code
> -------------------------------------------------
>
>                 Key: FALCON-595
>                 URL: https://issues.apache.org/jira/browse/FALCON-595
>             Project: Falcon
>          Issue Type: Bug
>            Reporter: Raghav Kumar Gautam
>            Assignee: Raghav Kumar Gautam
>              Labels: authorization, security
>         Attachments: falcon-595.patch
>
>
> I found some minor improvements for DefaultAuthorizationProvider and I am opening this bug as per discussion on:
> https://issues.apache.org/jira/browse/FALCON-464



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