nevi-me commented on pull request #9468: URL: https://github.com/apache/arrow/pull/9468#issuecomment-777440219
heads-up @alamb, the implementation of `NullArray::new_with_type` is not sound, because it's not compliant with the spec. A `NullArray` must always have `DataType::Null`. I was working on a solution for this last night, but didn't finish it; I'll wrap it up tonight. This mainly fixes #9331. This PR will still be necessary, as we'll need to be able to print `NullArray` correctly (but haven't looked at the code) ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
