kumarUjjawal commented on PR #21739: URL: https://github.com/apache/datafusion/pull/21739#issuecomment-4287962379
> This might just be personal preference speaking, but in a physical explain plan I'm looking for what the engine is actually doing (how is it being executed), not what I wrote as query. It doesn't make sense for me that the logical plan (which is more declarative in nature) shows the lowered version, while the physical plan (which is more imperative) does not. If anything it should be the other way around that logical hides this detail, and physical shows it. > > So if I got to choose I would prefer the left side plan of the diff image above rather than the right one. I think I like this approach too, showing lowered expressions in physical explain can be more useful. Let me see how the code looks for this. -- 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]
