You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by GitBox <gi...@apache.org> on 2022/03/22 21:56:16 UTC

[GitHub] [geode] upthewaterspout commented on pull request #7456: make ResourceManager's interaction with JVM customizable with the HeapUsageProvider SPI

upthewaterspout commented on pull request #7456:
URL: https://github.com/apache/geode/pull/7456#issuecomment-1075681555


   On general comment before I dig in a bit deeper - it looks like this will only work correctly if there is only 1 implementation of this interface on the classpath?
   
   It might be better to have a way for a user to choose which implementation of HeapUsageMonitor they want - or perhaps even for the HeapUsageMonitor to itself decide if it is relevant for the current JVM settings?


-- 
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: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org