pitrou commented on issue #50249: URL: https://github.com/apache/arrow/issues/50249#issuecomment-4851636855
> I'm okay with that, and I'm aware that `SparseUnion`, `DenseUnion`, and `RunEndEncoded` builders have the bitmap buffer in different positions, so I should use a different strategy to check whether the data is null. I'll send a follow-up PR to address this by adding `IsNull` support for all builders. Actually, I think we should only do it for Builders that have a top-level physical null bitmap (therefore, not REE or Unions). > 2. I found that the following code doesn't appear to be used anywhere, and its logic also seems incorrect. Shouldn't we remove it? Good catch, feel free to open an issue and PR then :) -- 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]
