[GitHub] [spark] otterc commented on pull request #32140: [SPARK-32922][SHUFFLE][CORE] Adds support for executors to fetch local and remote merged shuffle data

2021-06-29 Thread GitBox
otterc commented on pull request #32140: URL: https://github.com/apache/spark/pull/32140#issuecomment-870994802 Thanks @mridulm and @Ngone51 for the thorough reviews! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

[GitHub] [spark] otterc commented on pull request #32140: [SPARK-32922][SHUFFLE][CORE] Adds support for executors to fetch local and remote merged shuffle data

2021-06-29 Thread GitBox
otterc commented on pull request #32140: URL: https://github.com/apache/spark/pull/32140#issuecomment-870101000 -- 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 unsubscri

[GitHub] [spark] otterc commented on pull request #32140: [SPARK-32922][SHUFFLE][CORE] Adds support for executors to fetch local and remote merged shuffle data

2021-06-29 Thread GitBox
otterc commented on pull request #32140: URL: https://github.com/apache/spark/pull/32140#issuecomment-870315289 The test failures are unrelated. -- 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 t

[GitHub] [spark] otterc commented on pull request #32140: [SPARK-32922][SHUFFLE][CORE] Adds support for executors to fetch local and remote merged shuffle data

2021-06-28 Thread GitBox
otterc commented on pull request #32140: URL: https://github.com/apache/spark/pull/32140#issuecomment-870260954 > Sorry for the delay. I'll do a review today. BTW, are there any other necessary magnet PRs that have to be merged for the 3.2 cut/release? There are 2 pending tasks which

[GitHub] [spark] otterc commented on pull request #32140: [SPARK-32922][SHUFFLE][CORE] Adds support for executors to fetch local and remote merged shuffle data

2021-06-28 Thread GitBox
otterc commented on pull request #32140: URL: https://github.com/apache/spark/pull/32140#issuecomment-870101000 @mridulm Resolved the conflict. -- 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 th

[GitHub] [spark] otterc commented on pull request #32140: [SPARK-32922][SHUFFLE][CORE] Adds support for executors to fetch local and remote merged shuffle data

2021-06-20 Thread GitBox
otterc commented on pull request #32140: URL: https://github.com/apache/spark/pull/32140#issuecomment-864697969 This PR is not a WIP any more. @mridulm @Ngone51 I have addressed all the review comments so far. PTAL. -- This is an automated message from the Apache Git Service. To respond