Return-Path: Delivered-To: apmail-hadoop-hdfs-issues-archive@minotaur.apache.org Received: (qmail 93965 invoked from network); 21 Sep 2010 23:07:14 -0000 Received: from unknown (HELO mail.apache.org) (140.211.11.3) by 140.211.11.9 with SMTP; 21 Sep 2010 23:07:14 -0000 Received: (qmail 6142 invoked by uid 500); 21 Sep 2010 23:07:14 -0000 Delivered-To: apmail-hadoop-hdfs-issues-archive@hadoop.apache.org Received: (qmail 6080 invoked by uid 500); 21 Sep 2010 23:07:14 -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 6072 invoked by uid 99); 21 Sep 2010 23:07:14 -0000 Received: from Unknown (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 21 Sep 2010 23:07:14 +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; Tue, 21 Sep 2010 23:06:56 +0000 Received: from thor (localhost [127.0.0.1]) by thor.apache.org (8.13.8+Sun/8.13.8) with ESMTP id o8LN6YHn025887 for ; Tue, 21 Sep 2010 23:06:34 GMT Message-ID: <14503938.330781285110394716.JavaMail.jira@thor> Date: Tue, 21 Sep 2010 19:06:34 -0400 (EDT) From: "Boris Shkolnik (JIRA)" To: hdfs-issues@hadoop.apache.org Subject: [jira] Updated: (HDFS-1394) modify -format option for namenode to generated new blockpool id and accept newcluster In-Reply-To: <8995911.156891284400354232.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-1394?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Boris Shkolnik updated HDFS-1394: --------------------------------- Attachment: HDFS-1394-8.patch * Before Line 138: Is there anything that needs to be done for simulated storage? it is done in HDFS-1365 * DataStorage.java's only change is whitespace. This can be removed. Done. * Nit: FsImage:guessClusterId seems like an odd name. determineClusterId? Done. * Reading through guessClusterId I was initially confused as to if it were ok to return a null to indicate failure. Javadoc on the method would help. Done. * Line 279: if(!(System.in.read() == 'Y')) could be simplified to if(System.in.read != 'Y') Done. * In the tests there are quite a few calls to NameNode.clusterIdStr = "TestClusterId"; these should be refactored into a static method on GenericTestUtils.java. Done. * There are currently no tests that go over the -genclusterid paths. This could be tested in TestHDFSCLI It cannot be tested with TestHDFSCLI, because it assumes 'dfs' command and not 'namenode' * This changes user-interaction with the Namenode, so the forrest docs need to be updated as well. the relevent doc is in hadoop-common. Created HADOOP-6967 for this issue. > modify -format option for namenode to generated new blockpool id and accept newcluster > -------------------------------------------------------------------------------------- > > Key: HDFS-1394 > URL: https://issues.apache.org/jira/browse/HDFS-1394 > Project: Hadoop HDFS > Issue Type: Sub-task > Affects Versions: Federation Branch > Reporter: Boris Shkolnik > Assignee: Boris Shkolnik > Attachments: HDFS-1394-5.patch, HDFS-1394-6.patch, HDFS-1394-7.patch, HDFS-1394-7.patch, HDFS-1394-8.patch > > -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.