Skip to content

Comments

[MINOR] add test_createDataFrame_empty_partition in pyspark arrow tests#29398

Closed
WeichenXu123 wants to merge 1 commit intoapache:masterfrom
WeichenXu123:add_one_pyspark_arrow_test
Closed

[MINOR] add test_createDataFrame_empty_partition in pyspark arrow tests#29398
WeichenXu123 wants to merge 1 commit intoapache:masterfrom
WeichenXu123:add_one_pyspark_arrow_test

Conversation

@WeichenXu123
Copy link
Contributor

What changes were proposed in this pull request?

add test_createDataFrame_empty_partition in pyspark arrow tests

Why are the changes needed?

test edge cases.

Does this PR introduce any user-facing change?

no.

How was this patch tested?

N/A

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 if tests pass

@SparkQA
Copy link

SparkQA commented Aug 10, 2020

Test build #127276 has finished for PR 29398 at commit cda9d54.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@HyukjinKwon
Copy link
Member

Merged to master and branch-3.0.

HyukjinKwon pushed a commit that referenced this pull request Aug 10, 2020
### What changes were proposed in this pull request?
add test_createDataFrame_empty_partition in pyspark arrow tests

### Why are the changes needed?
test edge cases.

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

### How was this patch tested?
N/A

Closes #29398 from WeichenXu123/add_one_pyspark_arrow_test.

Authored-by: Weichen Xu <weichen.xu@databricks.com>
Signed-off-by: HyukjinKwon <gurwls223@apache.org>
(cherry picked from commit fc62d72)
Signed-off-by: HyukjinKwon <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.

3 participants