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

Kai Zheng commented on HDFS-8494:
---------------------------------

bq. BlockInfoStriped needs cellSize to calculate consumed space.
Did HDFS-8375 miss the change required here? What's the value that's used in 
existing codes?
bq. Is it necessary to pass ErasureCodingZone to BlockInfoStriped?
Please don't. If necessary please just pass {{cellSize}} along with schema as 
did in HDFS-8375.

[~vinayrpet], do we have anything to add here regarding the chunkSize/cellSize 
stuff here? 
I thought we could change {{HdfsConstants.BLOCK_STRIPED_CELL_SIZE}} to 
{{DEFAULT_BLOCK_STRIPED_CELL_SIZE}} and ensure it's only used when really 
necessary.

> Remove hard-coded chunk size in favor of ECZone
> -----------------------------------------------
>
>                 Key: HDFS-8494
>                 URL: https://issues.apache.org/jira/browse/HDFS-8494
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>    Affects Versions: HDFS-7285
>            Reporter: Kai Sasaki
>            Assignee: Kai Sasaki
>             Fix For: HDFS-7285
>
>
> It is necessary to remove hard-coded values inside NameNode configured in 
> {{HdfsConstants}}. In this JIRA, we can remove {{chunkSize}} gracefully in 
> favor of HDFS-8375.
> Because {{cellSize}} is now originally stored only in {{ErasureCodingZone}}, 
> {{BlockInfoStriped}} can receive {{ErasureCodingZone}} instead of 
> {{ECSchema}} when its initialization.



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

Reply via email to