Merge pull request #2465 from avshenuk/main

Fix a small bug with server metrics parsing + a few code cleanups.

Author: chernser

client-v2/src/main/java/com/clickhouse/client/api/data_formats/internal/ProcessParser.java

@@ -22,23 +22,23 @@ public class ProcessParser {
     };
 
     public static void parseSummary(String text, OperationMetrics metrics) {
-        Map<String, Integer> map = parse(text == null ? "{}" : text);
+        Map<String, Long> map = parse(text == null ? "{}" : text);
 
         for (ServerMetrics m : ServerMetrics.values()) {
             metrics.updateMetric(m, -1);
         }
 
         for (int i = 0; i < SUMMARY_FIELDS.length; i++) {
             String field = SUMMARY_FIELDS[i];
-            Integer value = map.get(field);
+            Long value = map.get(field);
             if (value != null) {
                 metrics.updateMetric(SUMMARY_METRICS[i], value);
             }
         }
     }
 
 
-    public static Map<String, Integer> parse(String json) {
+    public static Map<String, Long> parse(String json) {
         if (json == null) {
             throw new IllegalArgumentException("json is null");
         }
@@ -50,7 +50,7 @@ public static Map<String, Integer> parse(String json) {
             throw new IllegalArgumentException("JSON must start with '{' and end with '}'");
         }
 
-        Map<String, Integer> result = new HashMap<>();
+        Map<String, Long> result = new HashMap<>();
 
         String content = json.substring(1, json.length() - 1).trim();
         if (content.isEmpty()) {
@@ -79,7 +79,7 @@ public static Map<String, Integer> parse(String json) {
             }
 
             try {
-                int value = Integer.parseInt(valueStr);
+                long value = Long.parseLong(valueStr);
                 result.put(key, value);
             } catch (NumberFormatException e) {
                 // ignore error

client-v2/src/main/java/com/clickhouse/client/api/internal/Gauge.java

@@ -6,8 +6,8 @@ public class Gauge implements Metric {
 
     private volatile long value;
 
-    public Gauge(long readRows) {
-        this.value = readRows;
+    public Gauge(long value) {
+        this.value = value;
     }
 
     public void set(long value) {

client-v2/src/main/java/com/clickhouse/client/api/metrics/ClientMetrics.java

@@ -3,12 +3,12 @@
 public enum ClientMetrics {
 
     /**
-     * Operation duration in nanoseconds.
+     * Operation duration in milliseconds.
      */
     OP_DURATION("client.opDuration"),
 
     /**
-     * Duration of the operation serialization step in nanoseconds.
+     * Duration of the operation serialization step in milliseconds.
      */
     OP_SERIALIZATION("client.opSerialization");