Blizzara commented on issue #12244: URL: https://github.com/apache/datafusion/issues/12244#issuecomment-2699060966
I didn‘t see something in substrait.io where it‘d be explicitly said, but the example there (https://substrait.io/tutorial/sql_to_substrait/#types-and-schemas) does use NULLABILITY_REQUIRED. I don’t think it makes any difference in reality, given it’s the fields inside that struct type that matter, not the root struct itself. If the validator is happier with NULLABILITY_REQUIRED, fine by me to change it to that. -- 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]
