Github user srowen commented on the issue:

    https://github.com/apache/spark/pull/20259
  
    I don't think this number tells you the system is stable or not. If it did, 
how is it actionable to users who don't have any access to investigate further? 
    
    Certainly, you should have proper monitoring for all of these services, and 
their metrics, if you care. This bit of info just isn't informative compared to 
the large amount of metrics already exposed.
    
    (I could see exposing it as a metric though, if not already. That's what 
metrics reporting is for.)
    
    If users who have access can't do so because important logs disappear, that 
is certainly a log policy problem you need to fix. 
    
    Although it's "just one extra piece of info" I think there's no logic to 
adding just this piece info, and only here. I would oppose this change.


---

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

Reply via email to