Does synchronized block required for method "addMetric(String name, Metric metric)" in "AbstractMetricsContext" class?

[arjun12102019]

Why do we require explicit thread safety using synchronized block for "metrics" map in "AbstractMetricsContext" class.

metrics is concurrentHashmap and which will take care of thread safety by default for "get" and "put" operations.

Please confirm about my understanding, So that I will make changes and raise PR.