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 <[email protected]>
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
### 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 <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
---
.../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: [email protected]
For additional commands, e-mail: [email protected]
