pan3793 commented on PR #58702: URL: https://github.com/apache/spark/pull/58702#issuecomment-5644558369
Thanks, all four addressed in d0d6b52. 1. The description now names the fourth caller, and there are UPDATE and MERGE tests on a mixed partitioning through the row-level scan. Writing them surfaced that `RowLevelOperationTable` never delegated `partitioning()`, so `getPartitionPredicateSchema` has seen an empty partitioning for every row-level scan since SPARK-55596. That is a bug in its own right and applies to 4.2/4.3, so it is split out as #58756 (SPARK-59457); this PR is rebased on it and carries it as the first commit. 2. The post-scan `FilterExec` condition must now reference `id` and not `part_col`. 3. Renamed to `bucket-only partitioning: ...`. 4. The warning lists only the identity partition fields. -- 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]
