[GitHub] [druid] clintropolis commented on pull request #13698: merge druid-core, extendedset, and druid-hll into druid-processing to simplify everything

["clintropolis (via GitHub)" <gi...@apache.org>]

clintropolis commented on PR #13698:
URL: https://github.com/apache/druid/pull/13698#issuecomment-1426314987

   I made another minor adjustment to add a method to `JvmUtils`, `resetTestsToDefaultRuntimeInfo` which allows resetting the `RuntimeInfo` in the event that a test needed to static inject some overridden version for testing, such as https://github.com/apache/druid/blob/master/processing/src/test/java/org/apache/druid/query/DruidProcessingConfigTest.java#L67 which I think is the reason i initially had to switch some tests from using `Runtime.getRuntime().maxMemory()` to using `JvmUtils.getRuntimeInfo().getMaxHeapSizeBytes()` for them to pass. 


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