[GitHub] [spark] mridulm edited a comment on pull request #33078: [SPARK-35546][Shuffle] Enable push-based shuffle when multiple app attempts are enabled and manage concurrent access to the state in a better way

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

mridulm edited a comment on pull request #33078:
URL: https://github.com/apache/spark/pull/33078#issuecomment-871854550


   @Ngone51 I am leaving on slightly long-ish vacation without access to my laptop .. will it be possible for you to shepard this pr and #33034 ?
   These are the last two pending fixes for push based shuffle, and deal with correctness issues (and not functionality issues).
   If you feel we cant get them through, @otterc will have a pr for branch-3.2 ready which will disable push based shuffle for 3.2 - we can merge that instead to prevent issues for users trying to leverage push based shuffle in later versions.
   Thanks !


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