Use const digest JSON property name

Author: wendigo

core/trino-main/src/main/java/io/trino/server/ui/UiQueryResource.java

@@ -55,6 +55,7 @@
 import static com.fasterxml.jackson.annotation.JsonIgnoreProperties.Value.forIgnoredProperties;
 import static io.trino.connector.system.KillQueryProcedure.createKillQueryException;
 import static io.trino.connector.system.KillQueryProcedure.createPreemptQueryException;
+import static io.trino.plugin.base.metrics.TDigestHistogram.DIGEST_PROPERTY;
 import static io.trino.security.AccessControlUtil.checkCanKillQueryOwnedBy;
 import static io.trino.security.AccessControlUtil.checkCanViewQueryOwnedBy;
 import static io.trino.security.AccessControlUtil.filterQueries;
@@ -169,7 +170,7 @@ private JsonCodec<QueryInfo> buildQueryInfoCodec(ObjectMapper objectMapper)
                     .copy()
                     .setDefaultAttributes(attrs);
             // Don't serialize TDigestHistogram.digest which isn't useful and human readable
-            mapper.configOverride(TDigestHistogram.class).setIgnorals(forIgnoredProperties("digest"));
+            mapper.configOverride(TDigestHistogram.class).setIgnorals(forIgnoredProperties(DIGEST_PROPERTY));
 
             // Do not output @class property for metric types
             mapper.addMixIn(Metric.class, DropTypeInfo.class);

lib/trino-plugin-toolkit/src/main/java/io/trino/plugin/base/metrics/TDigestHistogram.java

@@ -33,6 +33,10 @@
 public class TDigestHistogram
         implements Distribution<TDigestHistogram>
 {
+    // This is important so that we can instruct Jackson to ignore this property
+    // in certain places (e.g. UiQueryResource)
+    public static final String DIGEST_PROPERTY = "digest";
+
     private final TDigest digest;
 
     public static TDigestHistogram fromValue(double value)
@@ -57,15 +61,15 @@ public synchronized TDigest getDigest()
         return TDigest.copyOf(digest);
     }
 
-    @JsonProperty("digest")
+    @JsonProperty(DIGEST_PROPERTY)
     public synchronized byte[] serialize()
     {
         return digest.serialize().getBytes();
     }
 
     @JsonCreator
     @DoNotCall
-    public static TDigestHistogram deserialize(@JsonProperty("digest") byte[] digest)
+    public static TDigestHistogram deserialize(@JsonProperty(DIGEST_PROPERTY) byte[] digest)
     {
         return new TDigestHistogram(TDigest.deserialize(wrappedBuffer(digest)));
     }