Remove redundant check

Grouping keys are always part of output symbols
trinodb · Oct 16, 2024 · bf536a8 · bf536a8
1 parent 77aca72
commit bf536a8
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/core/trino-main/src/main/java/io/trino/sql/planner/plan/AggregationNode.java b/core/trino-main/src/main/java/io/trino/sql/planner/plan/AggregationNode.java
@@ -31,7 +31,6 @@
 import io.trino.sql.planner.Symbol;
 import io.trino.type.FunctionType;
 
-import java.util.HashSet;
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
@@ -233,8 +232,7 @@ public boolean producesDistinctRows()
     {
         return aggregations.isEmpty() &&
                 !groupingSets.getGroupingKeys().isEmpty() &&
-                outputs.size() == groupingSets.getGroupingKeys().size() &&
-                new HashSet<>(outputs).containsAll(groupingSets.getGroupingKeys());
+                outputs.size() == groupingSets.getGroupingKeys().size();
     }
 
     public boolean isDecomposable(Session session, Metadata metadata)