Remove extra digest copy during Digest merge

trinodb · Mar 22, 2023 · c001664 · c001664
1 parent 1e5774d
commit c001664
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/lib/trino-plugin-toolkit/src/main/java/io/trino/plugin/base/metrics/TDigestHistogram.java b/lib/trino-plugin-toolkit/src/main/java/io/trino/plugin/base/metrics/TDigestHistogram.java
@@ -66,10 +66,15 @@ public synchronized TDigest getDigest()
     public TDigestHistogram mergeWith(TDigestHistogram other)
     {
         TDigest result = getDigest();
-        result.mergeWith(other.getDigest());
+        other.mergeTo(result);
         return new TDigestHistogram(result);
     }
 
+    private synchronized void mergeTo(TDigest digest)
+    {
+        digest.mergeWith(this.digest);
+    }
+
     @Override
     @JsonProperty
     public synchronized long getTotal()