I really don't like 'magic' code like this. In general, heuristics like this take some testing to get right and typically we'll never cover every eventuality. That means they need to allow overriding by setting some kind of system property which is also not ideal. In this particular case (and from the testing I've done with Classgraph) the number of threads has no correlation either with memory consumption or open file descriptors. Other than this bit, the rest LGTM.
[ Full content available at: https://github.com/apache/geode/pull/3124 ] This message was relayed via gitbox.apache.org for [email protected]
