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

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


The following commit(s) were added to refs/heads/master by this push:
     new 71a54f0b445 [MINOR][SQL][FOLLOWUP] Fix scalastyle in 
LimitPushDownThroughWindow
71a54f0b445 is described below

commit 71a54f0b4456028226d0867081e6becd4935d33f
Author: Dongjoon Hyun <dongj...@apache.org>
AuthorDate: Sun Mar 12 14:21:13 2023 -0700

    [MINOR][SQL][FOLLOWUP] Fix scalastyle in LimitPushDownThroughWindow
    
    ### What changes were proposed in this pull request?
    
    Although this is weird because of `[MINOR][SQL][FOLLOWUP]'. This PR aims to 
recover the CI failure on the master branch from the following commit.
    
    
https://github.com/apache/spark/commit/c75b3ab6f608037b6e210d60e29392f59d113a8a
    
    ### Why are the changes needed?
    
    - https://github.com/apache/spark/actions/runs/4397853036
    
    ![Screenshot 2023-03-12 at 1 23 49 
PM](https://user-images.githubusercontent.com/9700541/224571434-55a3b27f-619a-4c28-83b8-25c5657c7ee7.png)
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    Pass the CIs.
    
    Closes #40386 from dongjoon-hyun/SCALASTYLE.
    
    Authored-by: Dongjoon Hyun <dongj...@apache.org>
    Signed-off-by: Dongjoon Hyun <dongj...@apache.org>
---
 .../spark/sql/catalyst/optimizer/LimitPushDownThroughWindow.scala      | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/LimitPushDownThroughWindow.scala
 
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/LimitPushDownThroughWindow.scala
index b4174ab18a4..de1368d2816 100644
--- 
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/LimitPushDownThroughWindow.scala
+++ 
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/LimitPushDownThroughWindow.scala
@@ -30,7 +30,8 @@ import 
org.apache.spark.sql.catalyst.trees.TreePattern.{LIMIT, WINDOW}
  * }}}
  */
 object LimitPushDownThroughWindow extends Rule[LogicalPlan] {
-  // The window frame of Rank, DenseRank and RowNumber can only be UNBOUNDED 
PRECEDING to CURRENT ROW.
+  // The window frame of Rank, DenseRank and RowNumber can only be UNBOUNDED 
PRECEDING to
+  // CURRENT ROW.
   private def supportsPushdownThroughWindow(
       windowExpressions: Seq[NamedExpression]): Boolean = 
windowExpressions.forall {
     case Alias(WindowExpression(_: Rank | _: DenseRank | _: RowNumber, 
WindowSpecDefinition(Nil, _,


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

Reply via email to