Skip to content

Commit aa06eaa

Browse files
authored
Support prunning on the newly supported NullCount (#4990)
Currently panicks in some benchmarks, follows #4980. Signed-off-by: Adam Gutglick <[email protected]>
1 parent efa9f4d commit aa06eaa

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

vortex-file/src/pruning.rs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,10 @@ pub fn extract_relevant_file_stats_as_struct_row(
4545
let typed_stats = stat_set.as_typed_ref(&field_dtype);
4646

4747
for stat in stats {
48-
if stat == &Stat::Max || stat == &Stat::Min || stat == &Stat::NaNCount {
48+
if matches!(
49+
stat,
50+
Stat::Max | Stat::Min | Stat::NaNCount | Stat::NullCount
51+
) {
4952
let Some(stat_value) = typed_stats.get(*stat).and_then(|p| p.as_exact()) else {
5053
vortex_bail!("missing stat {}, {} from stats set", field, stat)
5154
};

0 commit comments

Comments
 (0)