Github user felixcheung commented on a diff in the pull request:

    https://github.com/apache/spark/pull/20050#discussion_r158438115
  
    --- Diff: 
core/src/test/scala/org/apache/spark/ExecutorAllocationManagerSuite.scala ---
    @@ -1063,6 +1067,9 @@ class ExecutorAllocationManagerSuite
             s"${sustainedSchedulerBacklogTimeout.toString}s")
           .set("spark.dynamicAllocation.executorIdleTimeout", 
s"${executorIdleTimeout.toString}s")
           .set("spark.dynamicAllocation.testing", "true")
    +      // SPARK-22864: effectively disable the allocation schedule by 
setting the period to a
    +      // really long value.
    +      .set(TESTING_SCHEDULE_INTERVAL_KEY, "10000")
    --- End diff --
    
    this is 10sec? it's not really that long...


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to