Convert MemoryAllocation to the record

Author: wendigo

core/trino-main/src/main/java/io/trino/memory/ClusterMemoryPool.java

@@ -157,14 +157,14 @@ private List<MemoryAllocation> mergeQueryAllocations(List<MemoryAllocation> left
         Map<String, MemoryAllocation> mergedAllocations = new HashMap<>();
 
         for (MemoryAllocation allocation : left) {
-            mergedAllocations.put(allocation.getTag(), allocation);
+            mergedAllocations.put(allocation.tag(), allocation);
         }
 
         for (MemoryAllocation allocation : right) {
             mergedAllocations.merge(
-                    allocation.getTag(),
+                    allocation.tag(),
                     allocation,
-                    (a, b) -> new MemoryAllocation(a.getTag(), a.getAllocation() + b.getAllocation()));
+                    (a, b) -> new MemoryAllocation(a.tag(), a.allocation() + b.allocation()));
         }
 
         return new ArrayList<>(mergedAllocations.values());

core/trino-spi/pom.xml

@@ -326,6 +326,49 @@
                                     <old>method java.util.Map&lt;io.trino.spi.QueryId, java.lang.Long&gt; io.trino.spi.memory.MemoryPoolInfo::getQueryMemoryRevocableReservations()</old>
                                     <justification>Removed unused field</justification>
                                 </item>
+                                <item>
+                                    <ignore>true</ignore>
+                                    <code>java.annotation.removed</code>
+                                    <old>parameter void io.trino.spi.memory.MemoryAllocation::&lt;init&gt;(===java.lang.String===, long)</old>
+                                    <new>parameter void io.trino.spi.memory.MemoryAllocation::&lt;init&gt;(===java.lang.String===, long)</new>
+                                    <annotation>@com.fasterxml.jackson.annotation.JsonProperty("tag")</annotation>
+                                    <justification>Class converted to a record</justification>
+                                </item>
+                                <item>
+                                    <ignore>true</ignore>
+                                    <code>java.annotation.removed</code>
+                                    <old>parameter void io.trino.spi.memory.MemoryAllocation::&lt;init&gt;(java.lang.String, ===long===)</old>
+                                    <new>parameter void io.trino.spi.memory.MemoryAllocation::&lt;init&gt;(java.lang.String, ===long===)</new>
+                                    <annotation>@com.fasterxml.jackson.annotation.JsonProperty("allocation")</annotation>
+                                    <justification>Class converted to a record</justification>
+                                </item>
+                                <item>
+                                    <ignore>true</ignore>
+                                    <code>java.annotation.removed</code>
+                                    <old>method void io.trino.spi.memory.MemoryAllocation::&lt;init&gt;(java.lang.String, long)</old>
+                                    <new>method void io.trino.spi.memory.MemoryAllocation::&lt;init&gt;(java.lang.String, long)</new>
+                                    <annotation>@com.fasterxml.jackson.annotation.JsonCreator</annotation>
+                                    <justification>Class converted to a record</justification>
+                                </item>
+                                <item>
+                                    <ignore>true</ignore>
+                                    <code>java.method.removed</code>
+                                    <old>method long io.trino.spi.memory.MemoryAllocation::getAllocation()</old>
+                                    <justification>Class converted to a record</justification>
+                                </item>
+                                <item>
+                                    <ignore>true</ignore>
+                                    <code>java.method.removed</code>
+                                    <old>method java.lang.String io.trino.spi.memory.MemoryAllocation::getTag()</old>
+                                    <justification>Class converted to a record</justification>
+                                </item>
+                                <item>
+                                    <ignore>true</ignore>
+                                    <code>java.class.kindChanged</code>
+                                    <old>class io.trino.spi.memory.MemoryAllocation</old>
+                                    <new>class io.trino.spi.memory.MemoryAllocation</new>
+                                    <justification>Class converted to a record</justification>
+                                </item>
                             </differences>
                         </revapi.differences>
                     </analysisConfiguration>

core/trino-spi/src/main/java/io/trino/spi/memory/MemoryAllocation.java

@@ -13,34 +13,12 @@
  */
 package io.trino.spi.memory;
 
-import com.fasterxml.jackson.annotation.JsonCreator;
-import com.fasterxml.jackson.annotation.JsonProperty;
-
 import static java.util.Objects.requireNonNull;
 
-public final class MemoryAllocation
+public record MemoryAllocation(String tag, long allocation)
 {
-    private final String tag;
-    private final long allocation;
-
-    @JsonCreator
-    public MemoryAllocation(
-            @JsonProperty("tag") String tag,
-            @JsonProperty("allocation") long allocation)
-    {
-        this.tag = requireNonNull(tag, "tag is null");
-        this.allocation = allocation;
-    }
-
-    @JsonProperty
-    public String getTag()
-    {
-        return tag;
-    }
-
-    @JsonProperty
-    public long getAllocation()
+    public MemoryAllocation
     {
-        return allocation;
+        requireNonNull(tag, "tag is null");
     }
 }