myegorov commented on PR #44631: URL: https://github.com/apache/arrow/pull/44631#issuecomment-2453801827
> I assumed we are going to wait till nullability annotations are properly integrated. I think I understand the value of requiring `@Nullable` checks, but wondering if the bugfix here should be blocked on the refactor? iiuc the `return null` in this PR is far from exceptional. The ability to make zero-copy slices of ValueVector and VectorSchemaRoot is important to our usage of Arrow. Do you think it's possible to fix the bug and file a task to forbid `null` by default as a fast follow? I'd be happy to look into this later but might require some rampup. -- 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]
