[native] Fix index join type conversion

zacw7 · zacw7 · commit c114770ebdfc · 2025-08-01T07:02:50.000-07:00
diff --git a/presto-native-execution/presto_cpp/main/types/PrestoToVeloxQueryPlan.cpp b/presto-native-execution/presto_cpp/main/types/PrestoToVeloxQueryPlan.cpp
@@ -1159,7 +1159,7 @@ core::JoinType toJoinType(protocol::JoinType type) {
     case protocol::JoinType::FULL:
       return core::JoinType::kFull;
     case protocol::JoinType::SOURCE_OUTER:
-      return core::JoinType::kInner; // TODO: Map to proper join type.
+      return core::JoinType::kLeft;
   }
 
   VELOX_UNSUPPORTED("Unknown join type");

Original file line number	Diff line number	Diff line change
`@@ -1159,7 +1159,7 @@ core::JoinType toJoinType(protocol::JoinType type) {`
`1159`	`1159`	`case protocol::JoinType::FULL:`
`1160`	`1160`	`return core::JoinType::kFull;`
`1161`	`1161`	`case protocol::JoinType::SOURCE_OUTER:`
`1162`		`- return core::JoinType::kInner; // TODO: Map to proper join type.`
	`1162`	`+ return core::JoinType::kLeft;`
`1163`	`1163`	`}`
`1164`	`1164`
`1165`	`1165`	`VELOX_UNSUPPORTED("Unknown join type");`