nevi-me commented on a change in pull request #9496:
URL: https://github.com/apache/arrow/pull/9496#discussion_r582560852



##########
File path: rust/arrow/src/buffer/immutable.rs
##########
@@ -36,7 +36,7 @@ use super::MutableBuffer;
 #[derive(Clone, PartialEq, Debug)]
 pub struct Buffer {
     /// the internal byte buffer.
-    data: Arc<Bytes>,
+    data: Arc<Bytes<u8>>,

Review comment:
       Is it worth still pursuing adding more information to `Buffer` while 
working separately on the proposal? I'm blocked on some parquet work that I'm 
doing, because it depends on being able to correctly slice nested arrays.




----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to