[
https://issues.apache.org/jira/browse/HDFS-8375?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14544983#comment-14544983
]
Vinayakumar B commented on HDFS-8375:
-------------------------------------
bq. 1. Maybe let's take this chance to update schema to ecSchema in
{HdfsFileStatus}}?
Sure I will make this change.
bq. With this change we should probably revisit the relationship
ErasureCodingInfo and ErasureCodingZoneInfo. If ErasureCodingInfo is to
represent all EC-related info for a file, then it should include cell size, and
this structure can be used to encapsulate all required info in places like
StripedBlockUtil.
Yes, I too felt same. Now ErasureCodingInfo and ErasureCodingZoneInfo carries
almost same information. of-course except {{cellSize}}. And morever
{{ErasureCodingInfo}} not really used anywhere, atleast till now. IMO, we can
remove it and use {{ErasureCodingZoneInfo}}. If something required in future we
can re-consider adding it. What you say?
This work can be done in another Jira IMO.
bq. I haven't yet finished reviewing the FSNamesystem changes, will complete my
review later today.
Sure, take your time. Meanwhile I will update patch for first comment.
> Add cellSize as an XAttr to ECZone
> ----------------------------------
>
> Key: HDFS-8375
> URL: https://issues.apache.org/jira/browse/HDFS-8375
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Reporter: Vinayakumar B
> Assignee: Vinayakumar B
> Attachments: HDFS-8375-HDFS-7285-01.patch
>
>
> Add {{cellSize}} as an Xattr for ECZone. as discussed
> [here|https://issues.apache.org/jira/browse/HDFS-8347?focusedCommentId=14539108&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14539108]
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
