You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@gluten.apache.org by fe...@apache.org on 2025/08/04 18:23:35 UTC
(incubator-gluten) branch main updated: [GLUTEN-10330] Correct the exception msg when making the Substrait plan (#10331)
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 <be...@163.com>
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: commits-unsubscribe@gluten.apache.org
For additional commands, e-mail: commits-help@gluten.apache.org