Move RebindSafeMBeanServer from Presto

martint / jmxutils

Exporting JMX mbeans made easy

Apache License 2.0

171 stars 47 forks source link

Closed takezoe closed 5 years ago

takezoe commented 5 years ago

Presto has RebindSafeMBeanServer in many sub projects, and I found TODO comment as:

// TODO: move this to airlift or jmxutils

While I'm not sure if jmxutils is a better place to move it than airlift, it good to remove this kind of code duplication anyway.

takezoe commented 5 years ago

Since another pull request (https://github.com/prestosql/presto/pull/669) is ongoing in Presto side, I close this pull request.