AngersZhuuuu 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 adding this information to events? Since for spark admin, we always want to build system to know the app's running status in our cluster and let user to change the memory configuration if they don't set it reasonable. So we nee to know the peak memory usage. Although we can get this information form metrics system but we need to integrate restful api's metrics data and metrics system's information. This pr make us can get driver's memory usage from hisitory server's restful api. -- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org