keith-turner opened a new issue, #4262: URL: https://github.com/apache/accumulo/issues/4262
The MeterRegistryFactory does not have any mechanism for passing configuration. Since its likely metrics will need configuration (like a server to send metrics to) this leads to placing the configuration for the MeterRegistryFactory class and any options needed by the class in different places. For example the MeterRegistryFactory class would be configured in accumulo.properties files and if it needs the hostname of a server this might be set as java system prop in accumulo-env.sh. This plugin could be modified to follow the pattern of many other SPI plugins for passing configuration to the plugin. Since a new type is being introduced in #4244, that presents a good opportunity to make this change. -- 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]
