FrankChen021 opened a new pull request, #20200: URL: https://github.com/apache/druid/pull/20200
Fixes #10615. ### Description The deprecated `org.apache.druid.java.util.metrics.SysMonitor` monitor is removed. Its no-op variant, unused Druid subclass, Guice provider, and obsolete tests are removed as well. `SigarUtil` now resolves its native library relative to itself because the deprecated monitor class is no longer available. Existing configurations that list `SysMonitor` in `druid.monitoring.monitors` must be updated to use `org.apache.druid.java.util.metrics.OshiSysMonitor`. The metrics documentation now documents only `OshiSysMonitor`, and the Druid 39 upgrade notes call out this incompatible removal. #### Release note `SysMonitor` has been removed after being deprecated in Druid 27. Use `OshiSysMonitor` instead. #### Validation - `mvn -ntp test -pl processing,server -am -Dtest="org.apache.druid.java.util.metrics.SigarLoadTest,org.apache.druid.java.util.metrics.OshiSysMonitorTest,org.apache.druid.java.util.metrics.NoopOshiSysMonitorTest,org.apache.druid.server.metrics.MetricsModuleTest" -Dsurefire.failIfNoSpecifiedTests=false -Pskip-static-checks -Dweb.console.skip=true -T1C` - `npm run spellcheck` -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
