[GitHub] [ozone] JacksonYao287 commented on pull request #3380: HDDS-6659: EC: Add BlockGroupLen info as part of PutBlock in EC Writes for helping in recovery.

[GitBox <gi...@apache.org>]

JacksonYao287 commented on PR #3380:
URL: https://github.com/apache/ozone/pull/3380#issuecomment-1118606082

   thanks @umamaheswararao for the work , the change overall looks good to me.
   i am not quite sure do we really need this info. when reading chunks of the last stripe in a block group,  if the size of a chunk is less than chunksize, (index-1) * chunksize + currentChunkSize will be the last stripe data size , so that we can get the block group length.
   
   do you mean if the last partial chunk is just in the lost container, without the block group lengh, we can not identify with part of the recovered chunk is data, and which part is padding?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org