viirya commented on PR #11977: URL: https://github.com/apache/datafusion/pull/11977#issuecomment-2297093270
> @viirya seems to suggest this would be a good change, but @jayzhan211 points out this change would mean anyone who built `PhysicalExpr`s without a corresponding `Expr` would have to create their own schema names Hmm, as we built physical operators directly (without going through analysis stage in DataFusion), we manually bind all columns. Do we still have additional name resolution in physical stage in DataFusion? -- 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
