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-39972][PYTHON][SQL][TESTS] Revert the test case of SPARK-39962 in branch-3.2 and branch-3.1 #37401

Closed
wants to merge 1 commit into from

Conversation

HyukjinKwon
Copy link
Member

What changes were proposed in this pull request?

This PR reverts the test in #37390 in branch-3.2 and branch-3.1 because testing util does not exist in branch-3.2 and branch-3.1.

Why are the changes needed?

See #37390 (comment)

Does this PR introduce any user-facing change?

No, test-only.

How was this patch tested?

Logically clean revert.

@HyukjinKwon HyukjinKwon changed the title [SPARK-39972] Revert the test case of SPARK-39962 in branch-3.2 and branch-3.1 [SPARK-39972][PYTHON][SQL][TESTS] Revert the test case of SPARK-39962 in branch-3.2 and branch-3.1 Aug 4, 2022
@HyukjinKwon
Copy link
Member Author

im gonna merge this as it's a clean revert of the test

Copy link
Contributor

@ulysses-you ulysses-you left a comment

Choose a reason for hiding this comment

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

thank you @HyukjinKwon

@HyukjinKwon
Copy link
Member Author

Merged to branch-3.2 and branch-3.1.

HyukjinKwon added a commit that referenced this pull request Aug 4, 2022
…in branch-3.2 and branch-3.1

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

This PR reverts the test in #37390 in branch-3.2 and branch-3.1 because testing util does not exist in branch-3.2 and branch-3.1.

### Why are the changes needed?

See #37390 (comment)

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

No, test-only.

### How was this patch tested?

Logically clean revert.

Closes #37401 from HyukjinKwon/SPARK-39972.

Authored-by: Hyukjin Kwon <gurwls223@apache.org>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
HyukjinKwon added a commit that referenced this pull request Aug 4, 2022
…in branch-3.2 and branch-3.1

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

This PR reverts the test in #37390 in branch-3.2 and branch-3.1 because testing util does not exist in branch-3.2 and branch-3.1.

### Why are the changes needed?

See #37390 (comment)

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

No, test-only.

### How was this patch tested?

Logically clean revert.

Closes #37401 from HyukjinKwon/SPARK-39972.

Authored-by: Hyukjin Kwon <gurwls223@apache.org>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
(cherry picked from commit bc9a5d7)
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
@HyukjinKwon HyukjinKwon closed this Aug 4, 2022
sunchao pushed a commit to sunchao/spark that referenced this pull request Jun 2, 2023
… in branch-3.2 and branch-3.1

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

This PR reverts the test in apache#37390 in branch-3.2 and branch-3.1 because testing util does not exist in branch-3.2 and branch-3.1.

### Why are the changes needed?

See apache#37390 (comment)

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

No, test-only.

### How was this patch tested?

Logically clean revert.

Closes apache#37401 from HyukjinKwon/SPARK-39972.

Authored-by: Hyukjin Kwon <gurwls223@apache.org>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
(cherry picked from commit bc9a5d7)
@HyukjinKwon HyukjinKwon deleted the SPARK-39972 branch January 15, 2024 00:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants