xudong963 commented on PR #24648: URL: https://github.com/apache/datafusion/pull/24648#issuecomment-5473410716
@jayzhan211 Thanks for catching this. Addressed in `e78603f50`: - Exact-empty files are excluded only from min/max reasoning and are appended back to rebuilt scan groups. - All-empty inputs preserve their original groups, avoiding `UnknownPartitioning(0)`. - The `file_index` / `file_count` mapping is removed, and `new_from_files` keeps its original empty-input error contract. - A shared `is_known_empty` helper is used by the callers. - Unit coverage now exercises both split variants and sort-within-group for mixed and all-empty inputs. - `parquet_sorted_statistics.slt` now covers the filtered empty-partition `GROUP BY` regression. I also ran the full workspace clippy/lint suite and the extended workspace tests, plus the targeted tests on the PR base. -- 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]
