[GitHub] [arrow-datafusion] xudong963 commented on a diff in pull request #4307: Add another api to create join plan

[GitBox <gi...@apache.org>]

xudong963 commented on code in PR #4307:
URL: https://github.com/apache/arrow-datafusion/pull/4307#discussion_r1028677521


##########
datafusion/expr/src/logical_plan/builder.rs:
##########
@@ -458,6 +461,81 @@ impl LogicalPlanBuilder {
         self.join_detailed(right, join_type, join_keys, filter, false)
     }
 
+    /// Apply a join with on constraint which the key is an expression.
+    ///
+    /// Filter expression expected to contain non-equality predicates that can not be pushed
+    /// down to any of join inputs.
+    /// In case of outer join, filter applied to only matched rows.
+    pub fn join_with_expr_keys(
+        &self,
+        right: &LogicalPlan,
+        join_type: JoinType,
+        join_keys: (Vec<impl Into<Expr>>, Vec<impl Into<Expr>>),

Review Comment:
   I think so. -- We should "believe" our optimizer, If it doesn't perform well, we can polish it!



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org