[GitHub] [spark] otterc commented on pull request #37052: [SPARK-39647][CORE] Register the executor with ESS before registering the BlockManager

otterc commented on PR #37052: URL: https://github.com/apache/spark/pull/37052#issuecomment-1180723189 All the tests pass now @Ngone51 @mridulm -- 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

[GitHub] [spark] otterc commented on pull request #37052: [SPARK-39647][CORE] Register the executor with ESS before registering the BlockManager

otterc commented on PR #37052: URL: https://github.com/apache/spark/pull/37052#issuecomment-1177877890 The same test in HealthTrackerIntegrationSuite failed again ``` Bad node with multiple executors, job will still succeed with the right confs *** FAILED *** (20 seconds, 47

[GitHub] [spark] otterc commented on pull request #37052: [SPARK-39647][CORE] Register the executor with ESS before registering the BlockManager

otterc commented on PR #37052: URL: https://github.com/apache/spark/pull/37052#issuecomment-1177035357 > @otterc GA failed This below test failed. I don't think it is related to this change. ``` info] *** 1 TEST FAILED *** [error] Failed: Total 3385, Failed 1, Errors 0,