Skip to content

Commit

Permalink
Parquet statistics: don't panic (#3127)
Browse files Browse the repository at this point in the history
We can always decide to read the parquet file and
do more work instead of panicking.
  • Loading branch information
ritchie46 committed Apr 12, 2022
1 parent 95096a7 commit b774caa
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion polars/polars-lazy/src/physical_plan/expressions/binary.rs
Original file line number Diff line number Diff line change
Expand Up @@ -393,7 +393,10 @@ mod stats {
let fld_l = self.left.to_field(schema)?;
let fld_r = self.right.to_field(schema)?;

assert_eq!(fld_l.data_type(), fld_r.data_type(), "implementation error");
debug_assert_eq!(fld_l.data_type(), fld_r.data_type(), "implementation error");
if fld_l.data_type() != fld_r.data_type() {
return Ok(true);
}

let dummy = DataFrame::new_no_checks(vec![]);
let state = ExecutionState::new();
Expand Down

0 comments on commit b774caa

Please sign in to comment.