[GitHub] [spark] tgravescs commented on a change in pull request #29002: [SPARK-32175][CORE] Fix the order between initialization for ExecutorPlugin and starting heartbeat thread

tgravescs commented on a change in pull request #29002: URL: https://github.com/apache/spark/pull/29002#discussion_r458816016 ## File path: core/src/main/scala/org/apache/spark/executor/Executor.scala ## @@ -227,6 +222,13 @@ private[spark] class Executor( metricsPoller.sta

[GitHub] [spark] tgravescs commented on a change in pull request #29002: [SPARK-32175][CORE] Fix the order between initialization for ExecutorPlugin and starting heartbeat thread

tgravescs commented on a change in pull request #29002: URL: https://github.com/apache/spark/pull/29002#discussion_r458814226 ## File path: core/src/test/scala/org/apache/spark/executor/ExecutorSuite.scala ## @@ -402,6 +403,73 @@ class ExecutorSuite extends SparkFunSuite a