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

    https://github.com/apache/spark/pull/21221#discussion_r188136532
  
    --- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala 
---
    @@ -1753,9 +1766,21 @@ class DAGScheduler(
         messageScheduler.shutdownNow()
         eventProcessLoop.stop()
         taskScheduler.stop()
    +    heartbeater.stop()
    +  }
    +
    +  /** Reports heartbeat metrics for the driver. */
    +  private def reportHeartBeat(): Unit = {
    --- End diff --
    
    It's a bit redundant for fields that aren't used by the driver -- for the 
driver, execution memory gets set to 0.


---

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

Reply via email to