[SLING-8764] A metric named commons.scheduler.oldest.running.job.millis already exists

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: Commons Scheduler 2.7.6
• Component/s: Commons
• Assignee: Stefan Egli
• Reporter: Miroslav Beranič
• Created: 03/Oct/19 22:20
• Updated: 10/Feb/20 09:53
• Resolved: 06/Feb/20 10:07

Description

GaugesSupport throws an exception when activation happens after first time, as metrics do not get unregistered from the MetricRegistry.

Method @Deactivate should be updated to unregister registered metric at the @Activate method.

There seems to be already some-kind of metric-name tracking, but do not know what is the useage. So I've created "quick&easy" solution, to track registered metric names and use this name array at the @Deactivate to unregister metric names.

 

I will provide PR at the project GitHub page located at the:

https://github.com/apache/sling-org-apache-sling-commons-scheduler

 

This is just a proposal, I guess author knows what is missing better.

Issue Links

• links to: GitHub Pull Request #1
• links to: GitHub Pull Request #2