Mailing-List: contact common-issues-help@hadoop.apache.org; run by ezmlm
Precedence: bulk
Date: Thu, 21 Dec 2017 05:02:00 +0000 (UTC)
From: "Heng Li (JIRA)" <jira@apache.org>
To: common-issues@hadoop.apache.org
Message-ID: <JIRA.12996643.1470928067000.508662.1513832520092@Atlassian.JIRA>
In-Reply-To: <JIRA.12996643.1470928067000@Atlassian.JIRA>
References: <JIRA.12996643.1470928067000@Atlassian.JIRA> <JIRA.12996643.1470928067976@jira-lw-us.apache.org>
Subject: [jira] [Commented] (HADOOP-13485) Log refactoring: method
 invocation should be replaced by variable in hadoop tools
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Thu, 21 Dec 2017 05:02:05 -0000


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

Heng Li commented on HADOOP-13485:
----------------------------------

Thanks [~xiaochen]. Yeah, I think it's better to combine them into one log line:
{code:java}
LOG.debug("NativeAzureFileSystem initializing. blockSize = {}", this.blockSize);
{code}
BTW, could you please add "henglicad" instead of my old username (my email address) as a contributor? I feel it's not good to spam the email address everywhere (sorry my mistake).
Hi [~vrushalic], would you update the patch? Otherwise I'll upload a patch here.


> Log refactoring: method invocation should be replaced by variable in hadoop tools
> ---------------------------------------------------------------------------------
>
>                 Key: HADOOP-13485
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13485
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: tools
>    Affects Versions: 2.7.2
>            Reporter: Nemo Chen
>              Labels: easy
>         Attachments: HADOOP-13485.001.patch
>
>
> Similar to the fix for HDFS-409. In file:
> hadoop-rel-release-2.7.2/hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azure/NativeAzureFileSystem.java
> {code:borderStyle=solid}
> this.blockSize = conf.getLong(AZURE_BLOCK_SIZE_PROPERTY_NAME,
>         MAX_AZURE_BLOCK_SIZE);
> if (LOG.isDebugEnabled()) {
>     LOG.debug("NativeAzureFileSystem. Initializing.");
>     LOG.debug("  blockSize  = "
>           + conf.getLong(AZURE_BLOCK_SIZE_PROPERTY_NAME, MAX_AZURE_BLOCK_SIZE));
> }
> {code}
> For simplicity and readability, the {{conf.getLong(AZURE_BLOCK_SIZE_PROPERTY_NAME, MAX_AZURE_BLOCK_SIZE))}} should be changed to {{this.blockSize}}.


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

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