Skip to content

Commit 960d6af

Browse files
allisonwang-dbdongjoon-hyun
authored andcommitted
[SPARK-33472][SQL][FOLLOW-UP] Update RemoveRedundantSorts comment
### What changes were proposed in this pull request? This PR is a follow-up for #30373 that updates the comment for RemoveRedundantSorts in QueryExecution. ### Why are the changes needed? To update an incorrect comment. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? N/A Closes #30584 from allisonwang-db/spark-33472-followup. Authored-by: allisonwang-db <66282705+allisonwang-db@users.noreply.github.com> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
1 parent b6b45bc commit 960d6af

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

sql/core/src/main/scala/org/apache/spark/sql/execution/QueryExecution.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -344,7 +344,7 @@ object QueryExecution {
344344
PlanSubqueries(sparkSession),
345345
RemoveRedundantProjects,
346346
EnsureRequirements,
347-
// `RemoveRedundantSorts` needs to be added before `EnsureRequirements` to guarantee the same
347+
// `RemoveRedundantSorts` needs to be added after `EnsureRequirements` to guarantee the same
348348
// number of partitions when instantiating PartitioningCollection.
349349
RemoveRedundantSorts,
350350
DisableUnnecessaryBucketedScan,

0 commit comments

Comments
 (0)