[GitHub] [spark] zhouyejoe commented on pull request #34326: [SPARK-37053][CORE] Add metrics to SparkHistoryServer

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

zhouyejoe commented on pull request #34326:
URL: https://github.com/apache/spark/pull/34326#issuecomment-1027353355


   > * If we don't set `spark.history.store.path`, it will use memory to store app info and it's not belong to LevelDB metrics, right?
   
   Yes, correct. In this case, it will only use the in memory key value store.
   
   > * Only when support hybrid store disk backend,  then we have in-memory LevelDBs, right? 
   
   I don't quite get the question here. When hybrid mode is enabled, the in-memory store used during parsing is called [InMemoryStore](https://github.com/apache/spark/blob/master/common/kvstore/src/main/java/org/apache/spark/util/kvstore/InMemoryStore.java#L45), not the actual LevelDB.
   
   


-- 
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.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

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