paleolimbot commented on issue #18060: URL: https://github.com/apache/datafusion/issues/18060#issuecomment-3412161745
I don't mind either way...I think a FieldRef will make it less likely that will accidentally drop metadata (which is very difficult to track down), at the expense of potentially breaking existing pub field access. This would also let a cast express nullability which may be helpful if it can be respected (is there another way to make a nullable field nullable or vice versa?). After I finish with the `Placeholder` update I'm happy to take a look at this (or feel free to start and I can review!) -- 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]
