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-unsubscr...@geode.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
