viirya commented on PR #5226: URL: https://github.com/apache/arrow-rs/pull/5226#issuecomment-1863645356
> Perhaps we could use StructArray::try_new which will validate nullability and return an error, if we aren't already. I think this would still preserve the invariant that nullable columns don't obtain nulls? > > Or to phrase it differently, the array constructors already verify this, so perhaps this is sufficient?? Ah, that's good to know `StructArray` already does the check. But currently the cast kernel doesn't use `try_new` but `new`. Let me update it and add corresponding test. Thanks. -- 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]
