-
Notifications
You must be signed in to change notification settings - Fork 28.1k
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
[SPARK-46009][SQL][FOLLOWUP] Remove unused PERCENTILE_CONT and PERCENTILE_DISC in g4 #46272
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1, LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, could you fix this, @beliefer ?
[info] *** 2 TESTS FAILED ***
[error] Failed: Total 7391, Failed 2, Errors 0, Passed 7389, Ignored 5
[error] Failed tests:
[error] org.apache.spark.sql.catalyst.SQLKeywordSuite
[error] (catalyst / Test / test) sbt.TestsFailedException: Tests unsuccessful
6629199
to
83664b1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems that we have more failures still, @beliefer . Could you take a look at all of them?
Gentle ping, @beliefer . |
83664b1
to
cc5ce4f
Compare
@dongjoon-hyun I'm a bit busy. Thank you for ping me. |
Thank you, @beliefer . |
@dongjoon-hyun @HyukjinKwon @yaooqinn Thank you! |
### What changes were proposed in this pull request? This PR propose to remove unused golden file. ### Why are the changes needed? #46272 removed unused `PERCENTILE_CONT` and `PERCENTILE_DISC` in g4. But I made a mistake and submitted my local test code. ### Does this PR introduce _any_ user-facing change? 'No'. ### How was this patch tested? GA ### Was this patch authored or co-authored using generative AI tooling? 'No'. Closes #46385 from beliefer/SPARK-46009_followup3. Authored-by: beliefer <beliefer@163.com> Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
### What changes were proposed in this pull request? This PR propose to remove unused golden file. ### Why are the changes needed? apache#46272 removed unused `PERCENTILE_CONT` and `PERCENTILE_DISC` in g4. But I made a mistake and submitted my local test code. ### Does this PR introduce _any_ user-facing change? 'No'. ### How was this patch tested? GA ### Was this patch authored or co-authored using generative AI tooling? 'No'. Closes apache#46385 from beliefer/SPARK-46009_followup3. Authored-by: beliefer <beliefer@163.com> Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
…TILE_DISC in g4 ### What changes were proposed in this pull request? This PR propose to remove unused `PERCENTILE_CONT` and `PERCENTILE_DISC` in g4 ### Why are the changes needed? apache#43910 merged the parse rule of `PercentileCont` and `PercentileDisc` into `functionCall`, but forgot to remove unused `PERCENTILE_CONT` and `PERCENTILE_DISC` in g4. ### Does this PR introduce _any_ user-facing change? 'No'. ### How was this patch tested? GA. ### Was this patch authored or co-authored using generative AI tooling? 'No'. Closes apache#46272 from beliefer/SPARK-46009_followup2. Authored-by: beliefer <beliefer@163.com> Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
### What changes were proposed in this pull request? This PR propose to remove unused golden file. ### Why are the changes needed? apache#46272 removed unused `PERCENTILE_CONT` and `PERCENTILE_DISC` in g4. But I made a mistake and submitted my local test code. ### Does this PR introduce _any_ user-facing change? 'No'. ### How was this patch tested? GA ### Was this patch authored or co-authored using generative AI tooling? 'No'. Closes apache#46385 from beliefer/SPARK-46009_followup3. Authored-by: beliefer <beliefer@163.com> Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
What changes were proposed in this pull request?
This PR propose to remove unused
PERCENTILE_CONT
andPERCENTILE_DISC
in g4Why are the changes needed?
#43910 merged the parse rule of
PercentileCont
andPercentileDisc
intofunctionCall
, but forgot to remove unusedPERCENTILE_CONT
andPERCENTILE_DISC
in g4.Does this PR introduce any user-facing change?
'No'.
How was this patch tested?
GA.
Was this patch authored or co-authored using generative AI tooling?
'No'.