ulysses-you commented on PR #58279: URL: https://github.com/apache/spark/pull/58279#issuecomment-5537966720
Thanks @peter-toth -- 12, 13 and 14 are addressed in 77c17e98254: * **12** -- the comment names the first-pass change verbatim, the user-facing section calls out the fallback's plan change, and the fallback test gained two bare first-pass arms: one where the pre-alignment split count and the old distinct-key count pick different sides (revert-verified), and one control where they agree. * **13** -- the repro no longer uses `ALTER TABLE`; it matches the test's `numRowsPerSplit = 1` shape, with the real-connector reading kept in one sentence. * **14** -- narrowed to `private[sql]`; the scaladoc states the stronger fact (the collection's `require` already unifies the count across all members, so the representative only decides whether a keyed member exists). Local suites green (204). -- 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]
