alamb commented on a change in pull request #1339:
URL: https://github.com/apache/arrow-datafusion/pull/1339#discussion_r757903433
##########
File path: datafusion/src/logical_plan/expr.rs
##########
@@ -508,6 +508,56 @@ impl Expr {
}
}
+ /// Return if Expr contains Expr::IsNull or Expr::IsNotNull
+ pub fn has_null_expr(&self) -> Result<bool> {
Review comment:
I'll check this PR out more carefully tomorrow. I still don't fully
understand the need for this function (as I don't think any column exprs can be
pushed down past the non preserved join side)
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
