hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vinayakumar B (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HDFS-8146) Protobuf changes for BlockECRecoveryCommand and its fields for making it ready for transfer to DN
Date Fri, 17 Apr 2015 11:29:59 GMT

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

Vinayakumar B commented on HDFS-8146:
-------------------------------------

I think, instead of having 2D arrays to have multiple ECRecovery tasks, 

1. we can have a separate PB message for {{BlockECRecoveryInfo}} 
2. In {{BlockECRecoveryCommand}}'s PB message we can have repeated field for {{BlockECRecoveryInfoProto}}.

bq. Similarly, should we use a DatanodeStorageInfo to represent each target? But that requires
adding PB support for DatanodeStorageInfo itself..
For the PB'fication of {{DatanodeStorageInfo}} we can have two separate PB fields, {{DatanodeInfoProto}}
and {{StorageTypeProto}}. Since this is just a target, we dont need other details in PB message.
Samething done in BlockCommand too.


> Protobuf changes for BlockECRecoveryCommand and its fields for making it ready for transfer
to DN 
> --------------------------------------------------------------------------------------------------
>
>                 Key: HDFS-8146
>                 URL: https://issues.apache.org/jira/browse/HDFS-8146
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Uma Maheswara Rao G
>            Assignee: Uma Maheswara Rao G
>         Attachments: HDFS-8146.0.patch
>
>
> As part of working on HDFS-8137, we need to prepare BlockECRecoveryCommand, BlockECRecoveryInfo,
DatanodeStorageInfo, DatanodeDescripter (We can use DatanodeInfo for proto trasfer)  should
be ready in proto format for transferring them in command. Since all this code could be straight
forward and to have better focussed review on core part, I propose to separate this part in
to this JIRA. First I will prepare all this supported classes protobuf ready and then trasnfer
them to DN as part of HDFS-8137 by including ECSchema.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message