Skip to content

[SPARK-43897][TESTS][PS][CONNECT] Reorganize ps.Series unit tests#41401

Closed
zhengruifeng wants to merge 1 commit intoapache:masterfrom
zhengruifeng:ps_tests_reorg_series
Closed

[SPARK-43897][TESTS][PS][CONNECT] Reorganize ps.Series unit tests#41401
zhengruifeng wants to merge 1 commit intoapache:masterfrom
zhengruifeng:ps_tests_reorg_series

Conversation

@zhengruifeng
Copy link
Contributor

What changes were proposed in this pull request?

Reorganize ps.Series unit tests, break test_series into several topics

Why are the changes needed?

for easy to maintain

Does this PR introduce any user-facing change?

No, test-only

How was this patch tested?

updated CI

@zhengruifeng zhengruifeng force-pushed the ps_tests_reorg_series branch from 4203ab9 to 8f25ddd Compare June 1, 2023 07:48
@zhengruifeng zhengruifeng changed the title [WIP][SPARK-43897][TESTS][PS][CONNECT] Reorganize ps.Series unit tests [SPARK-43897][TESTS][PS][CONNECT] Reorganize ps.Series unit tests Jun 1, 2023
@zhengruifeng zhengruifeng force-pushed the ps_tests_reorg_series branch 5 times, most recently from 2bec3e1 to 85b9feb Compare June 6, 2023 01:09
init

split again

Empty-Commit
@zhengruifeng zhengruifeng force-pushed the ps_tests_reorg_series branch from 85b9feb to 6f6bf5f Compare June 7, 2023 01:59
@zhengruifeng
Copy link
Contributor Author

merged to master

@zhengruifeng zhengruifeng deleted the ps_tests_reorg_series branch June 8, 2023 00:04
czxm pushed a commit to czxm/spark that referenced this pull request Jun 12, 2023
### What changes were proposed in this pull request?
Reorganize ps.Series unit tests, break `test_series` into several topics

### Why are the changes needed?
for easy to maintain

### Does this PR introduce _any_ user-facing change?
No, test-only

### How was this patch tested?
updated CI

Closes apache#41401 from zhengruifeng/ps_tests_reorg_series.

Authored-by: Ruifeng Zheng <ruifengz@apache.org>
Signed-off-by: Ruifeng Zheng <ruifengz@apache.org>
Sign up for free to 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

Comments