alamb edited a comment on pull request #1128: URL: https://github.com/apache/arrow-datafusion/pull/1128#issuecomment-947539287
> Yeah I think that's is a better approach. Those special cases with Expr::Not and Expr::BinaryExpr are not pretty in my implementation. Given that the vast majority of this PR can be deleted if we go that route I'm going to close this. Cool -- thank you @pjmore for pushing this forward. Would you like to keep working at it, or shall I polish up https://github.com/apache/arrow-datafusion/pull/1145 to get that merged? This feature is going to really push DataFusion forward I think -- 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-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
