adriangb commented on issue #14993: URL: https://github.com/apache/datafusion/issues/14993#issuecomment-3470290311
Here's one I'm not so sure of, open to feedback: https://github.com/apache/datafusion/pull/18386 The reason for this PR is that once you start moving things around it gets super messy. I'm hoping this change makes things a bit less messy by not having to `.expect()` on the schema in various places, but I also understand it's a lot of churn and may not be worth it; I can always find other ways to simplify the overall change. -- 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]
