[GitHub] [spark] mridulm commented on pull request #33615: [SPARK-36374][SHUFFLE][DOC] Push-based shuffle high level user documentation

mridulm commented on pull request #33615: URL: https://github.com/apache/spark/pull/33615#issuecomment-898627051 There is a linter error for `TransportConf`, can you take a look at it @venkata91 -- This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [spark] mridulm commented on pull request #33615: [SPARK-36374][SHUFFLE][DOC] Push-based shuffle high level user documentation

mridulm commented on pull request #33615: URL: https://github.com/apache/spark/pull/33615#issuecomment-898910495 GA is passing, and the jenkins failure looks unrelated. Looks like the PR should be in a reasonable state to merge assuming no other changes are required. Can you take a pass

[GitHub] [spark] mridulm commented on pull request #33615: [SPARK-36374][SHUFFLE][DOC] Push-based shuffle high level user documentation

mridulm commented on pull request #33615: URL: https://github.com/apache/spark/pull/33615#issuecomment-899595911 Given RC timelines, will it be possible to take a pass @Ngone51, @gengliangwang ? I want to merge only after you are fine with the latest version. Thanks. -- This is an autom

[GitHub] [spark] mridulm commented on pull request #33615: [SPARK-36374][SHUFFLE][DOC] Push-based shuffle high level user documentation

mridulm commented on pull request #33615: URL: https://github.com/apache/spark/pull/33615#issuecomment-899602206 Merged to master, branch-3.2 +CC @gengliangwang Thanks for working on this @venkata91 ! Thanks for the reviews @Ngone51 , @gengliangwang, @Victsm :-) -- This