gruuya commented on PR #25602: URL: https://github.com/apache/datafusion/pull/25602#issuecomment-5816222386
> Per file, the adapter often rewrites the probe child to CAST(b@0 AS Int64): a nullable table column over a REQUIRED parquet column, or an Int32 file under an Int64 table. The bare-Column check then drops the bitmap silently. On a 20-row-group file, b BIGINT kept 20/20 row groups while b BIGINT NOT NULL kept 4/20. Please accept same-type or widening integer casts; stat_column_expr already rewrites the cast onto b_min/b_max. Also add an slt case with a nullable table over REQUIRED data. Makes sense; added conditional casting unwrapping in both `build_hash_lookup_pruning_expr`/`min_max_non_null_exprs`, as well as some tests to exercise it. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
