You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2021/09/15 14:31:52 UTC

[GitHub] [ozone] umamaheswararao opened a new pull request #2648: HDDS-5477 : EC: commit key should consolidate and create one keyLocationInfo per blockGrp

umamaheswararao opened a new pull request #2648:
URL: https://github.com/apache/ozone/pull/2648


   ## What changes were proposed in this pull request?
   
   Consolidates the block stream information per the block group.
   
   ## What is the link to the Apache JIRA
   
   https://issues.apache.org/jira/browse/HDDS-5477
   
   ## How was this patch tested?
   
   To check current tests impact.
   


-- 
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


[GitHub] [ozone] sodonnel commented on pull request #2648: HDDS-5477 : EC: commit key should consolidate and create one keyLocationInfo per blockGrp

Posted by GitBox <gi...@apache.org>.
sodonnel commented on pull request #2648:
URL: https://github.com/apache/ozone/pull/2648#issuecomment-920878510


   Thanks for adding the tests etc. This change LGTM and has passed all checks so I will go ahead and commit it.


-- 
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


[GitHub] [ozone] umamaheswararao commented on pull request #2648: HDDS-5477 : EC: commit key should consolidate and create one keyLocationInfo per blockGrp

Posted by GitBox <gi...@apache.org>.
umamaheswararao commented on pull request #2648:
URL: https://github.com/apache/ozone/pull/2648#issuecomment-920566337


   @sodonnel Thanks for the review. I have added a test to validate commitKey info. I figured out that client was not sending in sorted order. Anyway while reading you would do that based on index order. However I have just did simple sort to send them in correct order. 


-- 
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


[GitHub] [ozone] sodonnel merged pull request #2648: HDDS-5477 : EC: commit key should consolidate and create one keyLocationInfo per blockGrp

Posted by GitBox <gi...@apache.org>.
sodonnel merged pull request #2648:
URL: https://github.com/apache/ozone/pull/2648


   


-- 
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