[GitHub] [druid] imply-cheddar commented on issue #13833: Druid doesn't explicitly support C4 garbage collector

["imply-cheddar (via GitHub)" <gi...@apache.org>]

imply-cheddar commented on issue #13833:
URL: https://github.com/apache/druid/issues/13833#issuecomment-1465433741

   This is really a problem with that test more than anything.  The names that C4 is generating are fine and would "normally" flow out through metrics, just that test really wants to have some indication of young vs. old so that it can try to decide when it should stop.  The test itself is dubious if you ask me (is it trying to validate that the JVM does GC?  Or trying to validate things that the monitor does.  Likely, the JvmMonitor itself should be getting its values from an interface (with an overloaded, default implementation that delegates to the MXBeans) and the test should be passing in its own definition of that interface to generate values that the test would validate.  The fragility of the test seems to be rooted in trying to validate that the JVM does what's on the tin, which is something that I don't think our tests should be attempting to validate.


-- 
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: commits-unsubscribe@druid.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org