Return-Path: Delivered-To: apmail-hadoop-hdfs-issues-archive@minotaur.apache.org Received: (qmail 31406 invoked from network); 30 Sep 2010 23:19:57 -0000 Received: from unknown (HELO mail.apache.org) (140.211.11.3) by 140.211.11.9 with SMTP; 30 Sep 2010 23:19:57 -0000 Received: (qmail 65151 invoked by uid 500); 30 Sep 2010 23:19:57 -0000 Delivered-To: apmail-hadoop-hdfs-issues-archive@hadoop.apache.org Received: (qmail 65089 invoked by uid 500); 30 Sep 2010 23:19:57 -0000 Mailing-List: contact hdfs-issues-help@hadoop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: hdfs-issues@hadoop.apache.org Delivered-To: mailing list hdfs-issues@hadoop.apache.org Received: (qmail 65081 invoked by uid 99); 30 Sep 2010 23:19:57 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 30 Sep 2010 23:19:57 +0000 X-ASF-Spam-Status: No, hits=-2000.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.22] (HELO thor.apache.org) (140.211.11.22) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 30 Sep 2010 23:19:55 +0000 Received: from thor (localhost [127.0.0.1]) by thor.apache.org (8.13.8+Sun/8.13.8) with ESMTP id o8UNJX8f006268 for ; Thu, 30 Sep 2010 23:19:33 GMT Message-ID: <4850078.488851285888773333.JavaMail.jira@thor> Date: Thu, 30 Sep 2010 19:19:33 -0400 (EDT) From: "Dmytro Molkov (JIRA)" To: hdfs-issues@hadoop.apache.org Subject: [jira] Commented: (HDFS-1155) getDatanodeReport can be moved to NameNodeProtocol from ClientProtocol In-Reply-To: <29022713.46291273858604707.JavaMail.jira@thor> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/jira/browse/HDFS-1155?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12916730#action_12916730 ] Dmytro Molkov commented on HDFS-1155: ------------------------------------- My original assumption is not entirely accurate. Instead of moving the method I would like to duplicate the definition in the NameNode protocol and only make changes to the Balancer code to use a single nameNode protocol to talk to the namenode. > getDatanodeReport can be moved to NameNodeProtocol from ClientProtocol > ---------------------------------------------------------------------- > > Key: HDFS-1155 > URL: https://issues.apache.org/jira/browse/HDFS-1155 > Project: Hadoop HDFS > Issue Type: Improvement > Reporter: Dmytro Molkov > Assignee: Dmytro Molkov > > Right now getDatanodeReport is being used in only two places in the HDFS code: the Balancer and DFSAdmin. And it is the only reason for these classes to use DFSClient. If we would move the method definition (or copy for now deprecating the old location) to the NameNode protocol DFSAdmin and Balancer will not rely on DFSClient anymore and will be cleaner. This will also help the Balancer to run cleaner against HDFS-599 changes. > Thoughts? -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.