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

felixybw 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 0c83f9886e [GLUTEN-10330] Correct the exception msg when making the 
Substrait plan (#10331)
0c83f9886e is described below

commit 0c83f9886ee43fd0ca4fd3d4b44addea43ed43a9
Author: Jiaan Geng <[email protected]>
AuthorDate: Tue Aug 5 02:23:31 2025 +0800

    [GLUTEN-10330] Correct the exception msg when making the Substrait plan 
(#10331)
    
    correct the exception msg when making the Substrait plan.
    Currently, Gluten doesn't have the ColumnarWholestageTransformer and 
doTansform is a typo
---
 .../main/java/org/apache/gluten/substrait/plan/PlanBuilder.java    | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git 
a/gluten-substrait/src/main/java/org/apache/gluten/substrait/plan/PlanBuilder.java
 
b/gluten-substrait/src/main/java/org/apache/gluten/substrait/plan/PlanBuilder.java
index 0853ee4719..114fe3ed82 100644
--- 
a/gluten-substrait/src/main/java/org/apache/gluten/substrait/plan/PlanBuilder.java
+++ 
b/gluten-substrait/src/main/java/org/apache/gluten/substrait/plan/PlanBuilder.java
@@ -23,6 +23,8 @@ import 
org.apache.gluten.substrait.extensions.FunctionMappingNode;
 import org.apache.gluten.substrait.rel.RelNode;
 import org.apache.gluten.substrait.type.TypeNode;
 
+import com.google.common.base.Preconditions;
+
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
@@ -57,9 +59,8 @@ public class PlanBuilder {
       List<String> outNames,
       TypeNode outputSchema,
       AdvancedExtensionNode extension) {
-    if (subCtx == null) {
-      throw new NullPointerException("ColumnarWholestageTransformer cannot 
doTansform.");
-    }
+    Preconditions.checkNotNull(
+        subCtx, "Cannot execute doTransform due to the SubstraitContext is 
null.");
     List<FunctionMappingNode> mappingNodes = new ArrayList<>();
 
     for (Map.Entry<String, Long> entry : 
subCtx.registeredFunction().entrySet()) {


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to