You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
So I need to add a new displayName field to the Metric constructor and then update all the enum values. Is this approach right? Do I need to look into any more files?
Yes, if you are not familiar with this issue, then here is some background. There is a plugin that runs during the build (mvn clean package should be enough to do it) that will generate the file core/target/generated-docs/metrics3.md. This markdown file at some point will be copied over to the accumulo-website repository (see apache/accumulo-website#442 for example). I'm proposing that we add a display name (e.g. Server Low Memory Indicator) instead of the metric name accumulo.detected.low.memory at the start of each metric. We should include the metric name in the body of each section. The plugin may need to be modified as well to use the display name as the title and to put the metric name in the body of each metric.
@DomGarguilo wrote the plugin I believe, so he should be able to answer questions about this as well.
Add a display name that can be used by the Monitor
The text was updated successfully, but these errors were encountered: