denis-chudov commented on code in PR #6282:
URL: https://github.com/apache/ignite-3/pull/6282#discussion_r2222730810


##########
modules/metrics/src/main/java/org/apache/ignite/internal/metrics/MetricRegistry.java:
##########
@@ -274,7 +275,7 @@ private void removeMetricSet(String srcName) {
      *
      * @param metricSets New map of metric sets that should be saved to new 
version of metric snapshot.
      */
-    private void updateMetricSnapshot(Map<String, MetricSet> metricSets) {
+    private void updateMetricSnapshot(SortedMap<String, MetricSet> metricSets) 
{

Review Comment:
   It was not obvious that there is sorted map which finally is used for 
exporting the metrics in lexicographical order. I made it more clear.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@ignite.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to