Delete "initializing = true" in ExecutorAllocationManager.reset

Author: witgo

core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala

@@ -249,14 +249,6 @@ private[spark] class ExecutorAllocationManager(
    * yarn-client mode when AM re-registers after a failure.
    */
   def reset(): Unit = synchronized {
-    /**
-     * When some tasks need to be scheduled and initial executor = 0, resetting the initializing
-     * field may cause it to not be set to false in yarn.
-     * SPARK-20079: https://issues.apache.org/jira/browse/SPARK-20079
-     */
-    if (maxNumExecutorsNeeded() == 0) {
-      initializing = true
-    }
     numExecutorsTarget = initialNumExecutors
     numExecutorsToAdd = 1
 

core/src/test/scala/org/apache/spark/ExecutorAllocationManagerSuite.scala

@@ -863,9 +863,6 @@ class ExecutorAllocationManagerSuite
     assert(!initializing(manager))
     manager.reset()
     assert(!initializing(manager))
-    sc.listenerBus.postToAll(SparkListenerStageCompleted(stageInfo))
-    manager.reset()
-    assert(initializing(manager))
   }
 
   test("reset the state of allocation manager") {