nevi-me edited a comment on pull request #9495: URL: https://github.com/apache/arrow/pull/9495#issuecomment-785108044
i've been away for over a week due to personal issues. When Jorge started work on this, we had discussed the need for `Bytes` or `Buffer` to know its byte width, so that we could correct our zero-copy slicing/offsets. I had taken a different approach to this PR (which was to directly track the byte_size in the Buffer struct), but because I made a lot of changes to the code base, it was difficult for me to reconcile changes. This PR seems to deal more with the allocator, so not yet necessarily with `Buffer`. I'll try to see what impact it has on my (draft) work, when it's merged. [EDIT: this is done in https://github.com/apache/arrow/pull/9496, which I'm now looking at] > The change set of this PR is essentially bf1a7eb I'm fine with getting this PR merged, I'm a bit overwhelmed by the detail, but looking at bf1a7eb has certainly helped me with reviewing the change. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
