This is an automated email from the ASF dual-hosted git repository.
tustvold pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-rs.git
The following commit(s) were added to refs/heads/master by this push:
new 46bbd7debe Removing redundant as casts (#5168)
46bbd7debe is described below
commit 46bbd7debe23f7974bec256223633e806ec7cf06
Author: Vrishabh <[email protected]>
AuthorDate: Tue Dec 5 23:30:26 2023 +0530
Removing redundant as casts (#5168)
---
parquet/src/column/reader.rs | 5 +----
parquet/src/encodings/rle.rs | 6 +++---
2 files changed, 4 insertions(+), 7 deletions(-)
diff --git a/parquet/src/column/reader.rs b/parquet/src/column/reader.rs
index adfcd63907..6c712ead62 100644
--- a/parquet/src/column/reader.rs
+++ b/parquet/src/column/reader.rs
@@ -607,10 +607,7 @@ fn parse_v1_level(
}
Encoding::BIT_PACKED => {
let bit_width = num_required_bits(max_level as u64);
- let num_bytes = ceil(
- (num_buffered_values as usize * bit_width as usize) as i64,
- 8,
- ) as usize;
+ let num_bytes = ceil(num_buffered_values as usize * bit_width as
usize, 8);
Ok((num_bytes, buf.slice(..num_bytes)))
}
_ => Err(general_err!("invalid level encoding: {}", encoding)),
diff --git a/parquet/src/encodings/rle.rs b/parquet/src/encodings/rle.rs
index 5807f6b9c5..5d91c1e53d 100644
--- a/parquet/src/encodings/rle.rs
+++ b/parquet/src/encodings/rle.rs
@@ -230,7 +230,7 @@ impl RleEncoder {
self.bit_writer.put_vlq_int(indicator_value as u64);
self.bit_writer.put_aligned(
self.current_value,
- bit_util::ceil(self.bit_width as i64, 8) as usize,
+ bit_util::ceil(self.bit_width as usize, 8),
);
self.num_buffered_values = 0;
self.repeat_count = 0;
@@ -524,8 +524,8 @@ impl RleDecoder {
self.bit_packed_left = ((indicator_value >> 1) * 8) as u32;
} else {
self.rle_left = (indicator_value >> 1) as u32;
- let value_width = bit_util::ceil(self.bit_width as i64, 8);
- self.current_value = bit_reader.get_aligned::<u64>(value_width
as usize);
+ let value_width = bit_util::ceil(self.bit_width as usize, 8);
+ self.current_value =
bit_reader.get_aligned::<u64>(value_width);
assert!(self.current_value.is_some());
}
true
