liamzwbao commented on PR #7984: URL: https://github.com/apache/arrow-rs/pull/7984#issuecomment-3123627002
Hi @jhorstmann @Dandandan, thank you very much for the review! I have addressed the comments. However, I noticed that this implementation will fail some tests because the capacity of `Vec` does not round up to multiple of 64 like [MutableBuffer](https://github.com/apache/arrow-rs/blob/16794ab14fa62ecf67de0da9460cc5752a9358f4/arrow-buffer/src/buffer/mutable.rs#L88) does. I have updated the tests in this PR for reference. Would appreciate suggestions to address this issue. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
