Return-Path: X-Original-To: apmail-hadoop-hdfs-issues-archive@minotaur.apache.org Delivered-To: apmail-hadoop-hdfs-issues-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id A968B9EB2 for ; Fri, 27 Jan 2012 18:26:35 +0000 (UTC) Received: (qmail 21206 invoked by uid 500); 27 Jan 2012 18:26:35 -0000 Delivered-To: apmail-hadoop-hdfs-issues-archive@hadoop.apache.org Received: (qmail 20819 invoked by uid 500); 27 Jan 2012 18:26:34 -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 20784 invoked by uid 99); 27 Jan 2012 18:26:33 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 27 Jan 2012 18:26:33 +0000 X-ASF-Spam-Status: No, hits=-2000.0 required=5.0 tests=ALL_TRUSTED,T_RP_MATCHES_RCVD X-Spam-Check-By: apache.org Received: from [140.211.11.116] (HELO hel.zones.apache.org) (140.211.11.116) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 27 Jan 2012 18:26:31 +0000 Received: from hel.zones.apache.org (hel.zones.apache.org [140.211.11.116]) by hel.zones.apache.org (Postfix) with ESMTP id D7FF9135007 for ; Fri, 27 Jan 2012 18:26:10 +0000 (UTC) Date: Fri, 27 Jan 2012 18:26:10 +0000 (UTC) From: "Suresh Srinivas (Commented) (JIRA)" To: hdfs-issues@hadoop.apache.org Message-ID: <157497444.1212.1327688770886.JavaMail.tomcat@hel.zones.apache.org> In-Reply-To: <427431125.81335.1327587055395.JavaMail.tomcat@hel.zones.apache.org> Subject: [jira] [Commented] (HDFS-2843) Rename protobuf message StorageInfoProto to NodeInfoProto 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-2843?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13194979#comment-13194979 ] Suresh Srinivas commented on HDFS-2843: --------------------------------------- bq. it's still information that's specific to "storage" in the general sense, although it's not specific to a single storage directory This information is used by secondary namenode, checkpointer, backup node as well. Does that still make it "storage"? bq. That said, what about during a distributed upgrade process which will change the layoutVersion? Distributed upgrade was used only once long back and is not in use any more. It is the regular upgrade that considers layout version and CTime etc. bq. Imagine a scenario where a DN dies during an upgrade while writing the VERSION files to its various storage directories, and hence only some subset of the directories get the updated VERSION file. Would we then need to know what layoutVersion is present in each storage directory separately? I haven't checked on how this case is handled - just thinking out loud. I am not proposing any change in the VERSION file. All I am saying is, call this part NodeInfo instead of StorageInfo. All this information is still present in all the data structures. I plan to cleanup the hierarchy of classes where StorageInfo is super class of bunch of other classes such DataStorage, BlockPoolSliceStorage, Storage etc. > Rename protobuf message StorageInfoProto to NodeInfoProto > --------------------------------------------------------- > > Key: HDFS-2843 > URL: https://issues.apache.org/jira/browse/HDFS-2843 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: data-node, name-node > Reporter: Suresh Srinivas > Assignee: Suresh Srinivas > Attachments: HDFS-2843.patch > > > StorageInfoProto has cTime, layoutVersion, namespaceID and clusterID. This is really information of a node that is part of the cluster, such as Namenode, Standby/Secondary/Backup/Checkpointer and datanodes. To reflect this, I want to rename it as NodeInfoProto from StorageInfoProto. -- 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