jackylee-ch opened a new pull request, #83: URL: https://github.com/apache/paimon-mosaic/pull/83
## Purpose `compare_values` returns `None` for any pair involving NaN, so a NaN bound can never be replaced. A FLOAT/DOUBLE row group whose first non-null value was NaN had both min and max pinned to NaN, making the range useless for row-group skipping. ## Changes - Let a non-NaN value replace a NaN bound. An all-NaN column keeps NaN bounds, so a row group with any non-null value still has both set, as `serialize_stats` requires. ## Tests - `test_stats_nan_does_not_pin_min_max`, `test_stats_all_nan_keeps_both_bounds_set`. - `cargo fmt --all -- --check`, `cargo clippy --all-targets --workspace -- -D warnings`, `cargo test --release --workspace`: 475 passed, 0 failed (interop cases skipped locally). -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
