pepijnve opened a new pull request, #18821: URL: https://github.com/apache/datafusion/pull/18821
## Which issue does this PR close? - None, break out PR of changes done in #17813 ## Rationale for this change In #17813 `GuaranteeRewriter` is used from the `datafusion_expr` crate. In order to enable this the type needed to be moved from `datafusion_optimizer` to `datafusion_expr`. Additionally, during the development of #17813 some latent bugs were discovered in `GuaranteeRewriter` that have been resolved. ## What changes are included in this PR? - Move `GuaranteeRewriter` to `datafusion_expr` - Fix two bugs where rewrites of 'between' expression would fail * when one of the bounds was untyped null * when the lower bound was greater than the upper bound ## Are these changes tested? - Existing tests updated - Tests added for bugfixes ## Are there any user-facing changes? No -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
