theirix commented on PR #10042: URL: https://github.com/apache/arrow-rs/pull/10042#issuecomment-4595410920
> Do we have a test case from the original issue? I think DataFusion was calling a parse code path in arrow-rs, so maybe look to add the test case there? (parsing a datatype from string, I mean) Not anything I am aware of- can introduce one later. Anyway, writing such a test will be much easier if we have `ArrayData::try_new_null` - should we? `ArrayData::new_null` already has a few unwraps inside -- 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]
