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

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

[~navis] The latest patch also has this change in 
SQLStdHiveAccessController.java to make admin role comparison case sensitive. 
But role names are not case sensitive in sql std auth mode (also documented in 
the wiki).
{code}
-        if (!HiveMetaStore.ADMIN.equalsIgnoreCase(role.getRoleName())) {
+        if (!HiveMetaStore.ADMIN.equals(role.getRoleName())) {
{code}

> DefaultHiveAuthorizationProvider should not initialize a new HiveConf
> ---------------------------------------------------------------------
>
>                 Key: HIVE-6437
>                 URL: https://issues.apache.org/jira/browse/HIVE-6437
>             Project: Hive
>          Issue Type: Bug
>          Components: Configuration
>    Affects Versions: 0.13.0
>            Reporter: Harsh J
>            Assignee: Navis
>            Priority: Trivial
>         Attachments: HIVE-6437.1.patch.txt, HIVE-6437.2.patch.txt, 
> HIVE-6437.3.patch.txt, HIVE-6437.4.patch.txt, HIVE-6437.5.patch.txt, 
> HIVE-6437.6.patch.txt
>
>
> During a HS2 connection, every SessionState got initializes a new 
> DefaultHiveAuthorizationProvider object (on stock configs).
> In turn, DefaultHiveAuthorizationProvider carries a {{new HiveConf(…)}} that 
> may prove too expensive, and unnecessary to do, since SessionState itself 
> sends in a fully applied HiveConf to it in the first place.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to