[GitHub] [spark] AngersZhuuuu commented on pull request #31992: [SPARK-34898][CORE] We should send SparkListenerExecutorMetricsUpdateEventLog of `driver` appropriately

[GitBox <gi...@apache.org>]

AngersZhuuuu commented on pull request #31992:
URL: https://github.com/apache/spark/pull/31992#issuecomment-809854277


   > Who is consuming this event in spark ?
   > Btw, if all you want is monitor jvm stats (outside of spark ui), use codahale integration instead ?
   
   `AppStatusListener` will consumer this 
   ```
    override def onExecutorMetricsUpdate(event: SparkListenerExecutorMetricsUpdate): Unit = {
   ```
   I know spark metrics frame work can have a monitor on this.  But when we want to build an application analysis system like Dr.elephant. We want to get app status from restful api, right? Driver's memory usage is an important  index too. 
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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