jorgecarleitao commented on pull request #8829: URL: https://github.com/apache/arrow/pull/8829#issuecomment-743720998
@nevi-me , I would really appreciate your help here: there are major problems emerging in the parquet crate when I try to remove an unsafe struct. The background of this change is that `FatPtr` is using `Buffer::capacity` and `Buffer::raw_data()` which is brittle and difficult to reason because a pointer is only valid within a container's `len`, not `capacity`. This is also causing the error that rust compiler safeguard us against when borrowing a mutable and immutable reference at the same time. This is blocking #8796 I verified that in all 4 tests the content passed to `ColumnReaderImpl::read_batch` is the same in master and after this PR, but for some reason the outcome of that call is different, which hints that there is some other assumption on that call beyond what we are telling the compiler about. ---------------------------------------------------------------- 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: [email protected]
