[RATIS-1748] RatisMetricRegistryImpl.getMetricName could be cached to reduce garbage

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.0.0, 2.5.0
• Component/s: metrics
• Assignee: Lari Hotari
• Reporter: Lari Hotari
• Created: 17/Nov/22 09:29
• Updated: 22/Nov/22 19:16
• Resolved: 22/Nov/22 19:14

Description

In allocation profiling with AsyncProfiler, I can see that RatisMetricRegistryImpl.getMetricName produces unnecessary garbage. It's better to cache the results. 

Issue Links

• links to: GitHub Pull Request #786