felipecrv commented on PR #41115: URL: https://github.com/apache/arrow/pull/41115#issuecomment-2048209002
> +1 for this, so it replace `MayHaveLogicalNulls()` and validity buffer check to just reject the type like "NA", and just check the validity buffer when it might contains physical null? In that case, it's better to call `MayHaveLogicalNulls()` directly. But if you're writing kernels to a subset of types and want to have a local guarantee that accessing the bitmap is enough, you use this check, access the bitmap directly, and never even call `IsValid`, `MayHaveLogicalNulls` etc. -- 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]
