XiangpengHao commented on PR #6154: URL: https://github.com/apache/arrow-rs/pull/6154#issuecomment-2256701705
I think this is a clever PR and is complementary to the `gc` approach. If I understand correctly, it replaces unused buffers with empty buffers so they can be released. There's a hierarchy of `gc`s: (1) replace unused buffer with empty buffer, without touching the view array (2) directly remove the unused buffers, but need to correct the buffer ids in the view array (3) complete gc, as implemented in the current `gc` function. This PR makes the first approach, which is transparent to users and has relatively low overhead. -- 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]
