Skip to content
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-40453][SPARK-41715][CONNECT][TESTS][FOLLOWUP] Skip freqItems doctest due to Scala 2.13 failure #39983

Closed
wants to merge 1 commit into from

Conversation

dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented Feb 13, 2023

What changes were proposed in this pull request?

This is a follow-up of #39947 to ignore freqItems doctest back.

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

@dongjoon-hyun dongjoon-hyun changed the title [SPARK-40453][SPARK-41715][CONNECT][TESTS] Skip freqItems doctest due to Scala 2.13 failure [SPARK-40453][SPARK-41715][CONNECT][TESTS][FOLLOWUP] Skip freqItems doctest due to Scala 2.13 failure Feb 13, 2023
Copy link
Member

@HyukjinKwon HyukjinKwon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @dongjoon-hyun

@HyukjinKwon
Copy link
Member

Merged to master and branch-3.4. Since we should monitor the scheduled jobs, and CI doesn't verify this change.

HyukjinKwon pushed a commit that referenced this pull request Feb 13, 2023
…octest due to Scala 2.13 failure

### What changes were proposed in this pull request?

This is a follow-up of #39947 to ignore `freqItems` doctest back.

### Why are the changes needed?

### Does this PR introduce _any_ user-facing change?

### How was this patch tested?

Closes #39983 from dongjoon-hyun/SPARK-40453.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
(cherry picked from commit d703808)
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
@dongjoon-hyun
Copy link
Member Author

Thank you, @HyukjinKwon ! I was editing the PR description but it's fine.

@dongjoon-hyun dongjoon-hyun deleted the SPARK-40453 branch February 13, 2023 00:44
snmvaughan pushed a commit to snmvaughan/spark that referenced this pull request Jun 20, 2023
…octest due to Scala 2.13 failure

### What changes were proposed in this pull request?

This is a follow-up of apache#39947 to ignore `freqItems` doctest back.

### Why are the changes needed?

### Does this PR introduce _any_ user-facing change?

### How was this patch tested?

Closes apache#39983 from dongjoon-hyun/SPARK-40453.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
(cherry picked from commit d703808)
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants