Rostislav Svoboda created WFWIP-131:
---------------------------------------
Summary: SmallRye Metrics - MetricsRegistryImpl - removal of entry from map
which is currently iterated
Key: WFWIP-131
URL:
https://issues.jboss.org/browse/WFWIP-131
Project: WildFly WIP
Issue Type: Bug
Components: MP Metrics
Reporter: Rostislav Svoboda
Assignee: Jeff Mesnil
SmallRye Metrics - MetricsRegistryImpl - removal of entry from map which is currently
iterated
https://github.com/smallrye/smallrye-metrics/blob/master/implementation/s...
Method {{removeMatching}} is iterating over {{metricMap}}. If there is a match {{remove}}
method is called. This method removes entry from {{metricMap}} and from {{metadataMap}}.
I don't like the fact that the entry is removed from the map by external method while
the map is currently iterated by the {{removeMatching}} method.
I think
https://docs.oracle.com/javase/8/docs/api/java/util/Iterator.html#remove-- should
be called in {{removeMatching}} for {{metricMap}} entry + invocation of
metadataMap.remove(entry.getKey()) + log mesasge;
--
This message was sent by Atlassian Jira
(v7.12.1#712002)