alamb commented on issue #21246: URL: https://github.com/apache/datafusion/issues/21246#issuecomment-5171437651
> I have a focused patch and regression test that preserve the output column index while expanding projection aliases. It covers both duplicate id positions and verifies that each filter is rewritten to its own original input column. If programmatic physical plans are in scope for this rule, I can open the PR. Well I think the question is "is the programatically created plan valid, or does it violate some (potentially unstated) invariant"? If there is some undocumented / unenforced invariant, we could potentially look into enforcing that / erroring rather than allowing -- 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]
