This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch branch-3.1
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.1 by this push:
     new b1c471b  [SPARK-33472][SQL][FOLLOW-UP] Update RemoveRedundantSorts 
comment
b1c471b is described below

commit b1c471b2613a812bacb29ff9f115ac7d24ad7dc9
Author: allisonwang-db <66282705+allisonwang...@users.noreply.github.com>
AuthorDate: Fri Dec 4 15:15:19 2020 -0800

    [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...@users.noreply.github.com>
    Signed-off-by: Dongjoon Hyun <dongj...@apache.org>
    (cherry picked from commit 960d6af75d5ef29b1efcf0d03e7db840270382e6)
    Signed-off-by: Dongjoon Hyun <dongj...@apache.org>
---
 .../src/main/scala/org/apache/spark/sql/execution/QueryExecution.scala  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/sql/core/src/main/scala/org/apache/spark/sql/execution/QueryExecution.scala 
b/sql/core/src/main/scala/org/apache/spark/sql/execution/QueryExecution.scala
index 040d1f36..0531dd2 100644
--- 
a/sql/core/src/main/scala/org/apache/spark/sql/execution/QueryExecution.scala
+++ 
b/sql/core/src/main/scala/org/apache/spark/sql/execution/QueryExecution.scala
@@ -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,


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to