SmallRye Metrics - MetricsRegistryImpl - removal of entry from map which is currently iterated

SmallRye Metrics - MetricsRegistryImpl - removal of entry from map which is currently iterated

https://github.com/smallrye/smallrye-metrics/blob/master/implementation/src/main/java/io/smallrye/metrics/MetricsRegistryImpl.java#L266

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;