westonpace commented on pull request #11441: URL: https://github.com/apache/arrow/pull/11441#issuecomment-951315539
@lidavidm My work on the more sophisticated method is coming along but rather than replace this I think it will be an addition to this. So there will be two ways of calculating the byte size. With that in mind I renamed the function to `TotalBufferSize` and added detection of duplicate buffers. This basically gives you the amount of RAM being kept alive by this datum (although there may be other owners of that RAM, so there is no guarantee removing the datum will free up that much RAM). I also simplified the test cases a bit by using ArrayFromJSON. -- 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]
