-
Notifications
You must be signed in to change notification settings - Fork 28k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[MINOR][SQL][FOLLOWUP] Fix scalastyle in LimitPushDownThroughWindow #40386
Conversation
cc @wangyum and @kenny-ddd |
Hi, @bjornjorgensen . Could you review this PR? |
Could you review this in order to recover master branch, @sunchao ? |
Merged to master. |
@dongjoon-hyun Thank you. Yes, this PR looks ok. |
Thank you so much, @bjornjorgensen . |
I have seen this one time before. Can you restart tests on the PR that fails? |
No, only the PR author can do that because we use the author's GitHub Actions for that PR.
For Apache Spark branch, of course, you can because you are the committer. |
Seams to be working now :) |
Thanks for your fix @dongjoon-hyun |
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.
c75b3ab
Why are the changes needed?
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass the CIs.