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 2022/12/03 07:28:56 UTC

[GitHub] [spark] mridulm commented on pull request #38560: [SPARK-38005][core] Support cleaning up merged shuffle files and state from external shuffle service

mridulm commented on PR #38560:
URL: https://github.com/apache/spark/pull/38560#issuecomment-1336104208

   Apologies for the delay in getting to this @yabola - will try to get to this next week.
   Thanks for your patience.
   
   My recommandation would be to keep the change as close to @wankunde's PR as possible, and fix the pending issues there to expedite the reviews (since that change was already reviewed quite a lot). If there are additional gaps in that PR, we can do it in follow up PR's.


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