fapifta commented on pull request #2702: URL: https://github.com/apache/ozone/pull/2702#issuecomment-941237977
As we discussed today, it seems that the ECBlockInputStream change is unnecessary, and shed light on a bug in the mock cluster side in case of mock based tests, where the original pipeline we got from the allocateBlock, did not contain replicaIndicies in the pipeline, while in a real cluster it is there for EC. I have removed the check from ECBlockInputStream, and added the indexes to the pipeline in the last commit. @umamaheswararao as we discussed probably this is the last change necessary, but please kindly do the review and let me know if you see anything else with the current patch. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
