alamb commented on issue #14618: URL: https://github.com/apache/datafusion/issues/14618#issuecomment-2660943935
> Good point [@alamb](https://github.com/alamb) . This would mean that the LogicalPlan, being the public API for _syntactial_ query building, is inherently not "fully resolved". Which suggests creating new IR might actually be an easier option that "healing" LP to become fully resolved IR itself. A new IR might work I think @wiedld and I were trying to do the "healing" (well really checking if the `LogicalPlan` isn't healthy) via the `InvariantLevel idea here: https://docs.rs/datafusion/latest/datafusion/logical_expr/enum.LogicalPlan.html#method.check_invariants It isn't yet sidely used, but maybe could be more and more formalized 🤔 -- 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...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
