You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "gaoyajun02 (via GitHub)" <gi...@apache.org> on 2023/09/20 03:35:33 UTC

[GitHub] [spark] gaoyajun02 commented on pull request #43004: [SPARK-45134][SHUFFLE] Avoid repeated fallback when failed to fetch remote push-merged block meta

gaoyajun02 commented on PR #43004:
URL: https://github.com/apache/spark/pull/43004#issuecomment-1726832193

   In fact, there is another fix, which is to add a check for outstandingRpcs in handleFailure of the network module.
   
   https://github.com/apache/spark/blob/227e262025229a67f43a8de452215053a9cbf662/common/network-common/src/main/java/org/apache/spark/network/client/TransportClient.java#L391
   
    but I am not sure whether it will affect other behaviors.
   
   


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