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

    https://github.com/apache/spark/pull/22473#discussion_r218941326
  
    --- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
    @@ -799,7 +799,8 @@ private[spark] class Executor(
           if (taskRunner.task != null) {
             taskRunner.task.metrics.mergeShuffleReadMetrics()
             taskRunner.task.metrics.setJvmGCTime(curGCTime - 
taskRunner.startGCTime)
    -        accumUpdates += ((taskRunner.taskId, 
taskRunner.task.metrics.accumulators()))
    +        accumUpdates +=
    +          ((taskRunner.taskId, 
taskRunner.task.metrics.accumulators().filterNot(_.isZero)))
    --- End diff --
    
    Could you add a flag for this behavior change?


---

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

Reply via email to