uros-b opened a new pull request, #58095: URL: https://github.com/apache/spark/pull/58095
### What changes were proposed in this pull request? This PR updates stale class and rule names in the adaptive query execution (AQE) test comments so they match the current planner. The referenced symbols were renamed or removed: - `LocalShuffleReaderExec` and `CoalescedShuffleReaderExec` were consolidated into `AQEShuffleReadExec`, which renders in query plans as `AQEShuffleRead (local)` and `AQEShuffleRead (coalesced)` (see `AQEShuffleReadExec.stringArgs`). The ASCII "possible resulting query plan" comments in `AdaptiveQueryExecSuite` are updated accordingly. - The pre-AQE `ExchangeCoordinator` was removed; post-shuffle partition coalescing is now performed by the `CoalesceShufflePartitions` rule. The comments in `CoalesceShufflePartitionsSuite` that attribute the post-shuffle partition estimate to the `ExchangeCoordinator` are retargeted to that rule. This is a comment-only change; no code is modified. ### Why are the changes needed? The comments reference classes that no longer exist in the codebase. `LocalShuffleReaderExec`, `CoalescedShuffleReaderExec`, and `ExchangeCoordinator` all have zero references under `sql/core/src/main`, so the comments misdescribe the plans these tests exercise and can mislead readers. ### Does this PR introduce _any_ user-facing change? No. Test-comment-only changes. ### How was this patch tested? No tests are required; the change touches comments only. The updated names were verified against the current source: `AQEShuffleReadExec.stringArgs` emits `local`/`coalesced`, and `CoalesceShufflePartitions` is the rule that estimates post-shuffle partition counts. Changed lines stay within 100 characters and introduce no non-ASCII characters. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Claude Code (Opus 4.8) -- 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]
