[GitHub] spark issue #20940: [SPARK-23429][CORE] Add executor memory metrics to heart...

[edwinalu <gi...@git.apache.org>]

Github user edwinalu commented on the issue:

    https://github.com/apache/spark/pull/20940
  
    @rezsafi, thanks for reviewing. It would be better if the heartbeat wasn't tied to sampling frequency. Most likely users would want to sample more frequently, although this would also mean more communication traffic between executors and driver.  I'm out of town right now, but am planning to reschedule the design discussion when I am back, hopefully in a couple of weeks, and will invite you.


---

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