hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Akira Ajisaka (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HDFS-11832) Switch leftover logs to slf4j format in BlockManager.java
Date Thu, 18 May 2017 17:01:04 GMT

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

Akira Ajisaka commented on HDFS-11832:
--------------------------------------

Some source code still need {{if(LOG.isDebugEnabled())}}. For example, the following source
code
{code}
  LOG.debug("blocks = {}", java.util.Arrays.asList(blocks));
{code}
creates additional ArrayList instance by {{Arrays.asList}} regardless of the log level, so
the code should be guarded.
This applies to 
{code}
      if (LOG.isDebugEnabled()) {
        LOG.debug("Initial report of block " + iblk.getBlockName()
            + " on " + storageInfo.getDatanodeDescriptor() + " size " +
            iblk.getNumBytes() + " replicaState = " + reportedState);
      }
{code}
as well.

> Switch leftover logs to slf4j format in BlockManager.java
> ---------------------------------------------------------
>
>                 Key: HDFS-11832
>                 URL: https://issues.apache.org/jira/browse/HDFS-11832
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: namenode
>    Affects Versions: 2.7.0, 2.8.0, 3.0.0-alpha1
>            Reporter: Hui Xu
>            Assignee: Chen Liang
>            Priority: Minor
>         Attachments: HDFS-11832.001.patch, HDFS-11832.002.patch, HDFS-11832.003.patch
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> HDFS-7706 Switch BlockManager logging to use slf4j. But the logging formats were not
modified appropriately. For example:
>       if (LOG.isDebugEnabled()) {
>         LOG.debug("blocks = " + java.util.Arrays.asList(blocks));
>       }
> These codes should be modified to:
>       LOG.debug("blocks = {}", java.util.Arrays.asList(blocks));



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-help@hadoop.apache.org


Mime
View raw message