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

    https://github.com/apache/spark/pull/22001#discussion_r209662081
  
    --- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala 
---
    @@ -402,6 +422,19 @@ class DAGScheduler(
         }
       }
     
    +  /**
    +   * Check whether the barrier stage requires more slots (to be able to 
launch all tasks in the
    +   * barrier stage together) than the total number of active slots 
currently. Fail current check
    +   * if trying to submit a barrier stage that requires more slots than 
current total number. If
    +   * the check fails consecutively for three times for a job, then fail 
current job submission.
    --- End diff --
    
    Seems I do not find the code about `"consecutively for three times"`, but 
only `maxFailureNumTasksCheck ` ?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to