Skip to content

[SPARK-42168][3.3][SQL][PYTHON][FOLLOW-UP] Test FlatMapCoGroupsInPandas with Window function - #39781

Closed
EnricoMi wants to merge 1 commit into
apache:branch-3.3from
G-Research-Forks:branch-3.3-cogroup-window-bug-test
Closed

[SPARK-42168][3.3][SQL][PYTHON][FOLLOW-UP] Test FlatMapCoGroupsInPandas with Window function#39781
EnricoMi wants to merge 1 commit into
apache:branch-3.3from
G-Research-Forks:branch-3.3-cogroup-window-bug-test

Conversation

@EnricoMi

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

This ports tests from #39717 in branch-3.2 to branch-3.3. See #39752 (comment).

Why are the changes needed?

To make sure this use case is tested.

Does this PR introduce any user-facing change?

No

How was this patch tested?

E2E test in test_pandas_cogrouped_map.py and analysis test in EnsureRequirementsSuite.scala.

…th Window function
This ports tests from apache#39717 in branch-3.2 to master.
To make sure this use case is tested.
No
E2E test in `test_pandas_cogrouped_map.py` and analysis test in `EnsureRequirementsSuite.scala`.
Closesapache#39752 from EnricoMi/branch-cogroup-window-bug-test.
Authored-by: Enrico Minack <github@enrico.minack.dev>
Signed-off-by: Chao Sun <sunchao@apple.com>
@HyukjinKwon

Copy link
Copy Markdown
Member

Thanks @EnricoMi

@HyukjinKwon

Copy link
Copy Markdown
Member

Merged to branch-3.3.

HyukjinKwon pushed a commit that referenced this pull request Jan 29, 2023
…as with Window function
### What changes were proposed in this pull request?
This ports tests from #39717 in branch-3.2 to branch-3.3. See #39752 (comment).
### Why are the changes needed?
To make sure this use case is tested.
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
E2E test in `test_pandas_cogrouped_map.py` and analysis test in `EnsureRequirementsSuite.scala`.
Closes#39781 from EnricoMi/branch-3.3-cogroup-window-bug-test.
Authored-by: Enrico Minack <github@enrico.minack.dev>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@EnricoMi@HyukjinKwon