Skip to content

Commit 16794ab

Browse files
authored
Minor: Restore warning comment on Int96 statistics read (#7975)
# Which issue does this PR close? - Follow on to #7687 # Rationale for this change I merged #7687 without addressing one of @emkornfield 's suggestions: https://github.com/apache/arrow-rs/pull/7687/files#r2205393903 # What changes are included in this PR? Implement the suggestion (restore a comment_ # Are these changes tested? BY CI # Are there any user-facing changes? No
1 parent 3e089d2 commit 16794ab

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

parquet/src/file/statistics.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -209,6 +209,7 @@ pub fn from_thrift(
209209
old_format,
210210
),
211211
Type::INT96 => {
212+
// INT96 statistics may not be correct, because comparison is signed
212213
let min = if let Some(data) = min {
213214
assert_eq!(data.len(), 12);
214215
Some(Int96::try_from_le_slice(&data)?)

0 commit comments

Comments
 (0)