From 2e682a2d70949707b3c7e0b1fcf0fa56e9af9709 Mon Sep 17 00:00:00 2001 From: David Sisson Date: Wed, 14 Aug 2024 22:16:34 -0700 Subject: [PATCH] Found a few more renames. --- .../apache/gluten/execution/CHHashJoinExecTransformer.scala | 2 +- .../gluten/execution/ShuffledHashJoinExecTransformer.scala | 4 ++-- cpp-ch/local-engine/Common/CHUtil.cpp | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/backends-clickhouse/src/main/scala/org/apache/gluten/execution/CHHashJoinExecTransformer.scala b/backends-clickhouse/src/main/scala/org/apache/gluten/execution/CHHashJoinExecTransformer.scala index 7080e55dc1863..c707ad988a53f 100644 --- a/backends-clickhouse/src/main/scala/org/apache/gluten/execution/CHHashJoinExecTransformer.scala +++ b/backends-clickhouse/src/main/scala/org/apache/gluten/execution/CHHashJoinExecTransformer.scala @@ -58,7 +58,7 @@ object JoinTypeTransform { case LeftSemi | ExistenceJoin(_) => JoinRel.JoinType.JOIN_TYPE_LEFT_SEMI case LeftAnti => - JoinRel.JoinType.JOIN_TYPE_ANTI + JoinRel.JoinType.JOIN_TYPE_LEFT_ANTI case _ => // TODO: Support cross join with Cross Rel JoinRel.JoinType.UNRECOGNIZED diff --git a/backends-velox/src/main/scala/org/apache/gluten/execution/ShuffledHashJoinExecTransformer.scala b/backends-velox/src/main/scala/org/apache/gluten/execution/ShuffledHashJoinExecTransformer.scala index d78e6c5b3e538..e3c93848dc2b6 100644 --- a/backends-velox/src/main/scala/org/apache/gluten/execution/ShuffledHashJoinExecTransformer.scala +++ b/backends-velox/src/main/scala/org/apache/gluten/execution/ShuffledHashJoinExecTransformer.scala @@ -69,7 +69,7 @@ case class ShuffledHashJoinExecTransformer( JoinRel.JoinType.JOIN_TYPE_LEFT_SEMI } case LeftAnti => - JoinRel.JoinType.JOIN_TYPE_ANTI + JoinRel.JoinType.JOIN_TYPE_LEFT_ANTI case _ => JoinRel.JoinType.UNRECOGNIZED } @@ -112,7 +112,7 @@ case class BroadcastHashJoinExecTransformer( case LeftSemi | ExistenceJoin(_) => JoinRel.JoinType.JOIN_TYPE_LEFT_SEMI case LeftAnti => - JoinRel.JoinType.JOIN_TYPE_ANTI + JoinRel.JoinType.JOIN_TYPE_LEFT_ANTI case _ => // TODO: Support cross join with Cross Rel JoinRel.JoinType.UNRECOGNIZED diff --git a/cpp-ch/local-engine/Common/CHUtil.cpp b/cpp-ch/local-engine/Common/CHUtil.cpp index 35b4f0c97806f..6ed353d3084cb 100644 --- a/cpp-ch/local-engine/Common/CHUtil.cpp +++ b/cpp-ch/local-engine/Common/CHUtil.cpp @@ -1106,7 +1106,7 @@ JoinUtil::getJoinKindAndStrictness(substrait::JoinRel_JoinType join_type, bool i return {DB::JoinKind::Left, DB::JoinStrictness::Any}; return {DB::JoinKind::Left, DB::JoinStrictness::Semi}; } - case substrait::JoinRel_JoinType_JOIN_TYPE_ANTI: + case substrait::JoinRel_JoinType_JOIN_TYPE_LEFT_ANTI: return {DB::JoinKind::Left, DB::JoinStrictness::Anti}; case substrait::JoinRel_JoinType_JOIN_TYPE_LEFT: return {DB::JoinKind::Left, DB::JoinStrictness::All};