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 eefe3d1931 [CORE] Remove createTransformContext function (#9714)
eefe3d1931 is described below
commit eefe3d1931513a19666ca1de41275fd6125327c2
Author: Mingliang Zhu <[email protected]>
AuthorDate: Thu May 22 13:33:00 2025 +0800
[CORE] Remove createTransformContext function (#9714)
---
.../execution/BroadcastNestedLoopJoinExecTransformer.scala | 8 +-------
.../org/apache/gluten/execution/JoinExecTransformer.scala | 8 +-------
.../main/scala/org/apache/gluten/execution/JoinUtils.scala | 14 --------------
.../gluten/execution/SortMergeJoinExecTransformer.scala | 4 +---
4 files changed, 3 insertions(+), 31 deletions(-)
diff --git
a/gluten-substrait/src/main/scala/org/apache/gluten/execution/BroadcastNestedLoopJoinExecTransformer.scala
b/gluten-substrait/src/main/scala/org/apache/gluten/execution/BroadcastNestedLoopJoinExecTransformer.scala
index 224dafefed..b9e124b608 100644
---
a/gluten-substrait/src/main/scala/org/apache/gluten/execution/BroadcastNestedLoopJoinExecTransformer.scala
+++
b/gluten-substrait/src/main/scala/org/apache/gluten/execution/BroadcastNestedLoopJoinExecTransformer.scala
@@ -159,13 +159,7 @@ abstract class BroadcastNestedLoopJoinExecTransformer(
inputStreamedOutput,
inputBuildOutput
)
-
- JoinUtils.createTransformContext(
- needSwitchChildren,
- output,
- projectRelPostJoinRel,
- inputStreamedOutput,
- inputBuildOutput)
+ TransformContext(output, projectRelPostJoinRel)
}
def validateJoinTypeAndBuildSide(): ValidationResult = {
diff --git
a/gluten-substrait/src/main/scala/org/apache/gluten/execution/JoinExecTransformer.scala
b/gluten-substrait/src/main/scala/org/apache/gluten/execution/JoinExecTransformer.scala
index 439b5689e8..8bac2752e3 100644
---
a/gluten-substrait/src/main/scala/org/apache/gluten/execution/JoinExecTransformer.scala
+++
b/gluten-substrait/src/main/scala/org/apache/gluten/execution/JoinExecTransformer.scala
@@ -274,13 +274,7 @@ trait HashJoinLikeExecTransformer extends BaseJoinExec
with TransformSupport {
)
context.registerJoinParam(operatorId, joinParams)
-
- JoinUtils.createTransformContext(
- needSwitchChildren,
- output,
- joinRel,
- inputStreamedOutput,
- inputBuildOutput)
+ TransformContext(output, joinRel)
}
def genJoinParameters(): Any = {
diff --git
a/gluten-substrait/src/main/scala/org/apache/gluten/execution/JoinUtils.scala
b/gluten-substrait/src/main/scala/org/apache/gluten/execution/JoinUtils.scala
index 12b544de90..5304aa1c46 100644
---
a/gluten-substrait/src/main/scala/org/apache/gluten/execution/JoinUtils.scala
+++
b/gluten-substrait/src/main/scala/org/apache/gluten/execution/JoinUtils.scala
@@ -302,20 +302,6 @@ object JoinUtils {
)
}
- def createTransformContext(
- exchangeTable: Boolean,
- output: Seq[Attribute],
- rel: RelNode,
- inputStreamedOutput: Seq[Attribute],
- inputBuildOutput: Seq[Attribute]): TransformContext = {
- val inputAttributes = if (exchangeTable) {
- inputBuildOutput ++ inputStreamedOutput
- } else {
- inputStreamedOutput ++ inputBuildOutput
- }
- TransformContext(output, rel)
- }
-
def createCrossRel(
substraitJoinType: CrossRel.JoinType,
condition: Option[Expression],
diff --git
a/gluten-substrait/src/main/scala/org/apache/gluten/execution/SortMergeJoinExecTransformer.scala
b/gluten-substrait/src/main/scala/org/apache/gluten/execution/SortMergeJoinExecTransformer.scala
index 3bca596940..227a14cbe3 100644
---
a/gluten-substrait/src/main/scala/org/apache/gluten/execution/SortMergeJoinExecTransformer.scala
+++
b/gluten-substrait/src/main/scala/org/apache/gluten/execution/SortMergeJoinExecTransformer.scala
@@ -223,10 +223,8 @@ abstract class SortMergeJoinExecTransformerBase(
)
context.registerJoinParam(operatorId, joinParams)
-
- JoinUtils.createTransformContext(false, output, joinRel,
inputStreamedOutput, inputBuildOutput)
+ TransformContext(output, joinRel)
}
-
}
/** Performs a sort merge join of two child relations. */
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
