Mailing-List: contact hdfs-issues-help@hadoop.apache.org; run by ezmlm
Precedence: bulk
Reply-To: hdfs-issues@hadoop.apache.org
Date: Fri, 18 Nov 2011 01:41:52 +0000 (UTC)
From: "Eli Collins (Commented) (JIRA)" <jira@apache.org>
To: hdfs-issues@hadoop.apache.org
Message-ID: 
 <1636202825.41659.1321580512581.JavaMail.tomcat@hel.zones.apache.org>
In-Reply-To: 
 <61493668.41237.1321577337249.JavaMail.tomcat@hel.zones.apache.org>
Subject: [jira] [Commented] (HDFS-2562) Refactor DN configuration variables
 out of DataNode class
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Eli Collins commented on HDFS-2562:
-----------------------------------

Looks good.  +1 pending hudson
                
> Refactor DN configuration variables out of DataNode class
> ---------------------------------------------------------
>
>                 Key: HDFS-2562
>                 URL: https://issues.apache.org/jira/browse/HDFS-2562
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: data-node
>    Affects Versions: 0.24.0, 0.23.1
>            Reporter: Todd Lipcon
>            Assignee: Todd Lipcon
>            Priority: Minor
>         Attachments: hdfs-2562.txt
>
>
> Right now there are many member variables in DataNode.java which are just read from configuration when the DN is started. Similar to what we did with DFSClient, we should refactor them into a new DNConf class which can be passed around - the motivation is to remove the many references we have throughout the code that read package-protected members of DataNode and reduce the number of members in DataNode itself.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira