AdamGS opened a new pull request, #27380: URL: https://github.com/apache/datafusion/pull/27380
## Which issue does this PR close? - Closes #24543. ## Rationale for this change It has been deprecated for a long time, I've ran into it looking into some other stuff, but I think its time. As I was working through some of this code, it seems like much of it was already effectively dead, with the relevant paths already handled elsewhere. I also couldn't figure out how to maintain full compatibility in the unparser test, I would love to hear how others feel about this change, it doesn't seem impossible I just don't have a strong intuition to what users expect here. ## What changes are included in this PR? 1. Removes the long deprecated `Expr::Wildcard`. 2. Remove support for it in both `datafusion-substrait` and `datafusion-proto`. ## What is the testing strategy for this PR? Existing tests. ## Are there any user-facing changes? Remove deprecated logical expr and associcated proto/substrait models. While the models haven't been marked as deprecated, the type itself has been for a long time, which seems like the main user-facing API. -- 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]
