Remove redundant clone method

The class is immutable.
trinodb · Mar 26, 2024 · 9e29231 · 9e29231
1 parent 6f6c668
commit 9e29231
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 7 deletions.
diff --git a/core/trino-main/src/main/java/io/trino/execution/resourcegroups/InternalResourceGroup.java b/core/trino-main/src/main/java/io/trino/execution/resourcegroups/InternalResourceGroup.java
@@ -954,7 +954,7 @@ public Collection<InternalResourceGroup> subGroups()
     ResourceUsage getResourceUsageSnapshot()
     {
         synchronized (root) {
-            return cachedResourceUsage.clone();
+            return cachedResourceUsage;
         }
     }
 

diff --git a/core/trino-main/src/main/java/io/trino/execution/resourcegroups/ResourceUsage.java b/core/trino-main/src/main/java/io/trino/execution/resourcegroups/ResourceUsage.java
@@ -32,12 +32,6 @@ public ResourceUsage(long cpuUsageMillis, long memoryUsageBytes)
         this.memoryUsageBytes = memoryUsageBytes;
     }
 
-    @Override
-    public ResourceUsage clone()
-    {
-        return new ResourceUsage(cpuUsageMillis, memoryUsageBytes);
-    }
-
     public ResourceUsage add(ResourceUsage other)
     {
         long newCpuUsageMillis = saturatedAdd(this.cpuUsageMillis, other.cpuUsageMillis);