subject:"\[GitHub\] \[spark\] LucaCanali commented on pull request #39127\: \[SPARK\-41585\]\[YARN\] The Spark exclude node functionality for YARN should work independently of dynamic allocation"

[GitHub] [spark] LucaCanali commented on pull request #39127: [SPARK-41585][YARN] The Spark exclude node functionality for YARN should work independently of dynamic allocation

2023-03-17 Thread via GitHub

LucaCanali commented on PR #39127: URL: https://github.com/apache/spark/pull/39127#issuecomment-1473415061 Thanks @tgraves , @attilapiros and @mridulm for reviewing this. I guess this is now ready to be merged to master? -- This is an automated message from the Apache Git Service. To

[GitHub] [spark] LucaCanali commented on pull request #39127: [SPARK-41585][YARN] The Spark exclude node functionality for YARN should work independently of dynamic allocation

2023-02-20 Thread via GitHub

LucaCanali commented on PR #39127: URL: https://github.com/apache/spark/pull/39127#issuecomment-1437434751 Thank you @tgravescs and @attilapiros for reviewing this. As for adding a test for this change, my first comm is that I see that existing tests already cover YARN_EXCLUDE_NODES