Skip to content

[SPARK-43896][TESTS][PS][CONNECT] Enable test_iterrows and test_itertuples on Connect#41400

Closed
zhengruifeng wants to merge 1 commit into
apache:masterfrom
zhengruifeng:enable_two_tests
Closed

[SPARK-43896][TESTS][PS][CONNECT] Enable test_iterrows and test_itertuples on Connect#41400
zhengruifeng wants to merge 1 commit into
apache:masterfrom
zhengruifeng:enable_two_tests

Conversation

@zhengruifeng
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

toLocalIterator had been implemented in https://issues.apache.org/jira/browse/SPARK-41876, this PR enables test_iterrows and test_itertuples on Connect

Why are the changes needed?

for better test coverage

Does this PR introduce any user-facing change?

No, test-only

How was this patch tested?

CI

@zhengruifeng
Copy link
Copy Markdown
Contributor Author

cc @itholic @HyukjinKwon

@HyukjinKwon
Copy link
Copy Markdown
Member

Merged to master.

@zhengruifeng zhengruifeng deleted the enable_two_tests branch June 1, 2023 07:10
czxm pushed a commit to czxm/spark that referenced this pull request Jun 12, 2023
…ertuples` on Connect

### What changes were proposed in this pull request?
`toLocalIterator` had been implemented in https://issues.apache.org/jira/browse/SPARK-41876, this PR enables `test_iterrows` and `test_itertuples` on Connect

### Why are the changes needed?
for better test coverage

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

### How was this patch tested?
CI

Closes apache#41400 from zhengruifeng/enable_two_tests.

Authored-by: Ruifeng Zheng <ruifengz@apache.org>
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

Development

Successfully merging this pull request may close these issues.

2 participants