[GitHub] [arrow-rs] tustvold commented on a diff in pull request #1683: Use bytes in parquet (#1474)

[GitBox <gi...@apache.org>]

tustvold commented on code in PR #1683:
URL: https://github.com/apache/arrow-rs/pull/1683#discussion_r868246096


##########
parquet/src/encodings/levels.rs:
##########
@@ -207,7 +207,7 @@ impl LevelDecoder {
                 let num_bytes =
                     ceil((num_buffered_values * bit_width as usize) as i64, 8);
                 let data_size = cmp::min(num_bytes as usize, data.len());
-                decoder.reset(data.range(data.start(), data_size));

Review Comment:
   This was actually a bug as data.range already takes into account the start offset



-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org