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

2021-06-15 Thread GitBox
AngersZh commented on pull request #31992: URL: https://github.com/apache/spark/pull/31992#issuecomment-862010798 > I don't know enough to have an opinion on this. I think the key questions are - what is the most consistent thing to do, and, are there any performance problems with

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

2021-04-06 Thread GitBox
AngersZh commented on pull request #31992: URL: https://github.com/apache/spark/pull/31992#issuecomment-814569934 > As I understood based on description and proposed changes, the right level of integration for this would be with metrics subsystem.. While we can do it via other means,

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

2021-04-01 Thread GitBox
AngersZh commented on pull request #31992: URL: https://github.com/apache/spark/pull/31992#issuecomment-811668969 ping @mridulm Any more update? -- 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

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

2021-03-29 Thread GitBox
AngersZh 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`