xgbtxy commented on code in PR #59732: URL: https://github.com/apache/spark/pull/59732#discussion_r4365466694
########## docs/sql-migration-guide.md: ########## @@ -24,6 +24,7 @@ license: | ## Upgrading from Spark SQL 4.4 to 5.0 +- Since Spark 5.0, Data Source V2 file scans preserve filters containing deterministic Python UDFs on partition columns for evaluation after the scan. Previously, these filters could be dropped, returning rows that did not satisfy the filter (SPARK-60486). Affected queries can now return fewer rows; data derived from their previous results may need to be recomputed. Review Comment: Updated in [2c9cf519](https://github.com/xgbtxy/spark/commit/2c9cf51932d005c3f62626bc974ce038b8e768af): moved the migration entry to Spark SQL 4.3-to-4.4 and used the suggested wording, including the partition-only scope. The PR description now also clarifies that built-in Parquet/ORC reads use V1 by default and that only reads routed to V2 are affected. This commit changes only the migration guide; `git diff --check` passes. The implementation commit's fork CI completed successfully (30 passed, 2 skipped). Revision prepared with Codex assistance. -- 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]
