You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2021/12/18 15:51:47 UTC

[GitHub] [spark] otterc commented on pull request #34934: [SPARK-37675][CORE][SHUFFLE] Return PushMergedRemoteMetaFailedFetchResult if no available push-merged block

otterc commented on pull request #34934:
URL: https://github.com/apache/spark/pull/34934#issuecomment-997221539


   Will take a look on Monday. It's unusual that there isn't even a single chunk because that means that no blocks were merged. If there aren't any blocks that were merged then there shouldn't be a push-merged block for the iterator to fetch at all. In this case there isn't any need to fetch the meta. Seems like that there could be a bug somewhere else where empty mergeStatus for a partition is getting created.


-- 
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: reviews-unsubscribe@spark.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org