WillAyd commented on PR #15041: URL: https://github.com/apache/arrow/pull/15041#issuecomment-1371244885
Gotcha - was going back in my head to thinking we wanted to measure the bytes allocated to the null array mask for all types, but if that's not the case makes sense on the fixed vs variable width types. I So then I think this is in a good spot? Not sure if we wanted to go crazy with templating versus branching in the dedicated string function to set the `bytes_processed`. And within the string function I am branching with a `array->null_bitmap() == nullptr)` condition - I didn't see a clear winner of that versus `array->null_count() > 0` in the code base so not sure that matters -- 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]
