Skip to content

Commit

Permalink
[SPARK-33472][SQL][FOLLOW-UP] Update RemoveRedundantSorts comment
Browse files Browse the repository at this point in the history
### 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>
(cherry picked from commit 960d6af)
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
  • Loading branch information
allisonwang-db authored and dongjoon-hyun committed Dec 4, 2020
1 parent ebfb6df commit b1c471b
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -344,7 +344,7 @@ object QueryExecution {
PlanSubqueries(sparkSession),
RemoveRedundantProjects,
EnsureRequirements,
// `RemoveRedundantSorts` needs to be added before `EnsureRequirements` to guarantee the same
// `RemoveRedundantSorts` needs to be added after `EnsureRequirements` to guarantee the same
// number of partitions when instantiating PartitioningCollection.
RemoveRedundantSorts,
DisableUnnecessaryBucketedScan,
Expand Down

0 comments on commit b1c471b

Please sign in to comment.