[
https://issues.apache.org/jira/browse/SENTRY-1663?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15927150#comment-15927150
]
Hao Hao commented on SENTRY-1663:
---------------------------------
+1 Thanks [~Jan Hentschel] for your contribution!
> UpdateableAuthzPermissions has mutable static fields
> ----------------------------------------------------
>
> Key: SENTRY-1663
> URL: https://issues.apache.org/jira/browse/SENTRY-1663
> Project: Sentry
> Issue Type: Bug
> Components: Sentry
> Affects Versions: 1.8.0
> Reporter: Alexander Kolbasov
> Assignee: Jan Hentschel
> Priority: Minor
> Labels: bite-sized, newbie
> Attachments: SENTRY-1663.001.patch
>
>
> UpdateableAuthzPermissions has the following code:
> {code}
> public static final Map<String, FsAction> ACTION_MAPPING = new
> HashMap<String, FsAction>();
>
> static {
> ACTION_MAPPING.put("ALL", FsAction.ALL);
> ACTION_MAPPING.put("*", FsAction.ALL);
> ACTION_MAPPING.put("SELECT", FsAction.READ_EXECUTE);
> ACTION_MAPPING.put("select", FsAction.READ_EXECUTE);
> ACTION_MAPPING.put("INSERT", FsAction.WRITE_EXECUTE);
> ACTION_MAPPING.put("insert", FsAction.WRITE_EXECUTE);
> }
> {code}
> It should use ImmutableMap instead.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
