xudong963 commented on issue #22883: URL: https://github.com/apache/datafusion/issues/22883#issuecomment-5902432849
> [@xudong963](https://github.com/xudong963) I looked into positional re-association. The conclusion I came to is that it is a pre-existing footgun in `main` (e.g. you reply `Yes` for the wrong filter and it's dropped). There is no good way to solve it, ultimately consumers need to manipulate these filters. No API is both restrictive enough to prevent errors but also flexible enough to accommodate all consumers. Using an `id` of sorts to track filters doesn't really solve it either. So I'd say let's not worry about it for now, wdyt? Thanks for looking into this. That makes sense: the positional contract already exists for PushedDown, so I agree that solving it generally should not block #25760. -- 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]
