Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21898#discussion_r207758008
  
    --- Diff: core/src/main/scala/org/apache/spark/BarrierTaskContext.scala ---
    @@ -80,7 +101,45 @@ class BarrierTaskContext(
       @Experimental
       @Since("2.4.0")
       def barrier(): Unit = {
    -    // TODO SPARK-24817 implement global barrier.
    +    val callSite = Utils.getCallSite()
    +    logInfo(s"Task $taskAttemptId from Stage $stageId(Attempt 
$stageAttemptNumber) has entered " +
    +      s"the global sync, current barrier epoch is $barrierEpoch.")
    +    logTrace(s"Current callSite: $callSite")
    --- End diff --
    
    +1


---

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

Reply via email to