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

Elek, Marton commented on HADOOP-16067:
---------------------------------------

+1. I will commit it to the trunk, soon.

Thank you [~charanh] the contribution.

> Incorrect Format Debug Statement KMSACLs
> ----------------------------------------
>
>                 Key: HADOOP-16067
>                 URL: https://issues.apache.org/jira/browse/HADOOP-16067
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: kms
>    Affects Versions: 3.2.0
>            Reporter: BELUGA BEHR
>            Assignee: Charan Hebri
>            Priority: Trivial
>              Labels: newbie, noob
>         Attachments: HADOOP-16067.001.patch
>
>
> {code:java}
>       if (LOG.isDebugEnabled()) {
>         LOG.debug("Checking user [{}] for: {}: {}" + ugi.getShortUserName(),
>             opType.toString(), acl.getAclString());
>       }
> {code}
> The logging message here is incorrect because the first variable is being 
> concatenated to the string instead of being passed as an argument.
> {code:java}
> -- Notice the user name 'hdfs' at the end and the spare curly brackets
> 2019-01-23 13:27:45,244 DEBUG 
> org.apache.hadoop.crypto.key.kms.server.KMSACLs: Checking user [GENERATE_EEK] 
> for: hdfs supergroup: {}hdfs
> {code}
> [https://github.com/apache/hadoop/blob/a55d6bba71c81c1c4e9d8cd11f55c78f10a548b0/hadoop-common-project/hadoop-kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSACLs.java#L313-L316]



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to