hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Zhe Zhang (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HDFS-8728) Erasure coding: revisit and simplify BlockInfoStriped and INodeFile
Date Tue, 14 Jul 2015 05:51:04 GMT

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

Zhe Zhang commented on HDFS-8728:

Thanks [~szetszwo] for raising the discussion. We have quite comprehensively compared the
2 {{BlockInfo}} hierarchies under HDFS-8499. As a recap, the current HDFS-7285 structure breaks
{{BIUC}} from {{BlockInfo}} which causes some intrusive changes to trunk (some codes assume
{{BIUC}} is a subclass of {{BlockInfo}}), which are not actually related to striping / EC.
Moreover, we are duplicating a pretty large chunk of code in {{BIContigousUC}} and {{BIStripedUC}}.
The new hierarchy requires some additional work on the branch, but it quite significantly
simplifies and clarifies the overall branch-trunk change.

To add to the analysis, I think the long term solution to this multi-inheritance problem is
to turn {{BIUC}} into a _feature_ / _op_ class as well. Since the EC consolidated patch is
already extra large, I think we should do that refactor separately.

> Erasure coding: revisit and simplify BlockInfoStriped and INodeFile
> -------------------------------------------------------------------
>                 Key: HDFS-8728
>                 URL: https://issues.apache.org/jira/browse/HDFS-8728
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Zhe Zhang
>            Assignee: Zhe Zhang
>         Attachments: HDFS-8728-HDFS-7285.00.patch, HDFS-8728-HDFS-7285.01.patch, HDFS-8728.00.patch,
HDFS-8728.01.patch, HDFS-8728.02.patch, Merge-1-codec.patch, Merge-2-ecZones.patch, Merge-3-blockInfo.patch,
Merge-4-blockmanagement.patch, Merge-5-blockPlacementPolicies.patch, Merge-6-locatedStripedBlock.patch,
Merge-7-replicationMonitor.patch, Merge-8-inodeFile.patch

This message was sent by Atlassian JIRA

View raw message