venkata91 commented on PR #3834: URL: https://github.com/apache/celeborn/pull/3834#issuecomment-6284655788
Thanks @zaynt4606 @SteNicholas for the review. Both remaining points are addressed in 000ec58: 1. The `NEVER` shuffle-tracking recommendation is removed from the runtime logs; the prerequisites for disabling tracking (Spark 3.5+ with `CelebornShuffleDataIO`, or shuffle decommissioning) are documented in the README and deployment guide instead. 2. Spotless formatting is fixed. CI is green except `spark3 (8, 3.4, LocalDiskShuffleDataIO)`, which failed in the known-flaky push-timeout integration tests (`pushMergeData timeout w/ replicate = true`, `pushdata timeout will add to pushExcludedWorkers`; see CELEBORN-1571 / #3876). These tests are unrelated to this change, and the same Spark 3.4 tests pass on Java 11 and 17. Could a committer re-trigger that job? PTAL, 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
