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

Aihua Xu commented on HIVE-17912:
---------------------------------

The change looks good to me. +1

Typically we need to add the if statement to avoid unnecessary expression 
evaluation in the LOG.trace() call, but it's not the case here.

> org.apache.hadoop.hive.metastore.security.DBTokenStore - Parameterize Logging
> -----------------------------------------------------------------------------
>
>                 Key: HIVE-17912
>                 URL: https://issues.apache.org/jira/browse/HIVE-17912
>             Project: Hive
>          Issue Type: Improvement
>          Components: HiveServer2
>    Affects Versions: 3.0.0
>            Reporter: BELUGA BEHR
>            Assignee: BELUGA BEHR
>            Priority: Trivial
>         Attachments: HIVE-17912.1.patch
>
>




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

Reply via email to