You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@gluten.apache.org by mi...@apache.org on 2025/09/17 15:31:15 UTC

(incubator-gluten) branch main updated: [GLUTEN-10701] Refactor SortRelNode to avoid addAll calls (#10702)

This is an automated email from the ASF dual-hosted git repository.

mingliang pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-gluten.git


The following commit(s) were added to refs/heads/main by this push:
     new 0a2ad39087 [GLUTEN-10701] Refactor SortRelNode to avoid addAll calls (#10702)
0a2ad39087 is described below

commit 0a2ad390875cdb13746e8977eaa1550145b9ad03
Author: Jiaan Geng <be...@163.com>
AuthorDate: Wed Sep 17 23:31:11 2025 +0800

    [GLUTEN-10701] Refactor SortRelNode to avoid addAll calls (#10702)
---
 .../src/main/java/org/apache/gluten/substrait/rel/SortRelNode.java | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/gluten-substrait/src/main/java/org/apache/gluten/substrait/rel/SortRelNode.java b/gluten-substrait/src/main/java/org/apache/gluten/substrait/rel/SortRelNode.java
index 90a541fa6f..ec7368cd3a 100644
--- a/gluten-substrait/src/main/java/org/apache/gluten/substrait/rel/SortRelNode.java
+++ b/gluten-substrait/src/main/java/org/apache/gluten/substrait/rel/SortRelNode.java
@@ -24,23 +24,22 @@ import io.substrait.proto.SortField;
 import io.substrait.proto.SortRel;
 
 import java.io.Serializable;
-import java.util.ArrayList;
 import java.util.List;
 
 public class SortRelNode implements RelNode, Serializable {
   private final RelNode input;
-  private final List<SortField> sorts = new ArrayList<>();
+  private final List<SortField> sorts;
   private final AdvancedExtensionNode extensionNode;
 
   public SortRelNode(RelNode input, List<SortField> sorts, AdvancedExtensionNode extensionNode) {
     this.input = input;
-    this.sorts.addAll(sorts);
+    this.sorts = sorts;
     this.extensionNode = extensionNode;
   }
 
   public SortRelNode(RelNode input, List<SortField> sorts) {
     this.input = input;
-    this.sorts.addAll(sorts);
+    this.sorts = sorts;
     this.extensionNode = null;
   }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@gluten.apache.org
For additional commands, e-mail: commits-help@gluten.apache.org