attilapiros commented on a change in pull request #24132: [SPARK-27189][CORE] 
Add executor-level memory usage metrics to the metrics system
URL: https://github.com/apache/spark/pull/24132#discussion_r266523874
 
 

 ##########
 File path: docs/monitoring.md
 ##########
 @@ -1006,6 +1009,37 @@ when running in local mode.
 
 - namespace=JVMCPU
   - jvmCpuTime
+  
+- namespace=MemoryMetrics
+  - **note:** MemoryMetrics counters are updated as part the executor metrics 
heart 
 
 Review comment:
   Nit: In the `executor metrics heart beat` I do not think the `metrics` word 
is correct there it is `executor's heartbeat` (I have only seen heartbeat 
written as one word i.e. at the config description of 
[spark.executor.heartbeatInterval](https://spark.apache.org/docs/latest/configuration.html)).
 

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

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

Reply via email to