amol- commented on pull request #10713: URL: https://github.com/apache/arrow/pull/10713#issuecomment-884295728
> I'm not opposed to this a priori, but I would find it better if the test suite avoided creating invalid arrays (except where specifically desired, e.g. to test validation). I wasn't much concerned about the test suite, which I agree can create valid arrays (creating null arrays smells like a form of premature optimization). I think that my focus was more toward C++ users ending up with a `nullptr` returned from somewhere else. In such case it makes your life a little easier if you just get printed out that the array is invalid instead of having to attach `gdb` to understand why you are facing a crash -- 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]
