hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Arpit Agarwal (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HDFS-5417) Fix storage IDs in PBHelper and UpgradeUtilities
Date Fri, 25 Oct 2013 01:58:01 GMT

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

Arpit Agarwal commented on HDFS-5417:
-------------------------------------

Hi Nicholas,

Should we fix the reverse convert routine for consistency?

{code}
  public static LocatedBlock convert(LocatedBlockProto proto) {
...
    LocatedBlock lb = new LocatedBlock(PBHelper.convert(proto.getB()), targets,
        proto.getStorageIDsList().toArray(new String[proto.getStorageIDsCount()]),
{code}

to be something like:
{code}
    LocatedBlock lb = new LocatedBlock(PBHelper.convert(proto.getB()), targets,
     proto.getStorageIDsCount() == 0 ? null : proto.getStorageIDsList().toArray(new String[proto.getStorageIDsCount()]),
{code}

+1 otherwise.

I filed HDFS-5423 to verify other initializations of LocatedBlock/RecoveringBlock to see if
we missed StorageIDs.

Thanks.

> Fix storage IDs in PBHelper and UpgradeUtilities
> ------------------------------------------------
>
>                 Key: HDFS-5417
>                 URL: https://issues.apache.org/jira/browse/HDFS-5417
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Tsz Wo (Nicholas), SZE
>            Assignee: Tsz Wo (Nicholas), SZE
>            Priority: Minor
>         Attachments: h5417_20131024.patch
>
>
> - PBHelper should check null for storage IDs when converting LocatedBlock.
> - UpgradeUtilities should createStorageID in createDataNodeVersionFile(..).



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Mime
View raw message