alamb commented on code in PR #9593:
URL: https://github.com/apache/arrow-rs/pull/9593#discussion_r2997942362
##########
arrow-buffer/src/buffer/mutable.rs:
##########
@@ -450,7 +450,13 @@ impl MutableBuffer {
/// Clear all existing data from this buffer.
pub fn clear(&mut self) {
- self.len = 0
+ self.len = 0;
+ #[cfg(feature = "pool")]
Review Comment:
this sees unrelated (though a good change)
I think we should either have a test for this or remove it from the PR
(maybe it would be good as a follow on PR -- I am not sure if there are other
places where something similar should be)
--
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]
