Skip to content

Conversation

@yikf
Copy link
Contributor

@yikf yikf commented Apr 17, 2023

What changes were proposed in this pull request?

This PR aims to improve spark.sql.files.minPartitionNum's doc.

Why are the changes needed?

Improve description

Does this PR introduce any user-facing change?

Yes, better config description.

How was this patch tested?

GA

@yikf
Copy link
Contributor Author

yikf commented Apr 17, 2023

cc @ulysses-you

@yikf yikf force-pushed the config-minPartitionNum-doc branch 2 times, most recently from c0123d0 to c44c0ab Compare April 17, 2023 12:55
@amaliujia
Copy link
Contributor

Looks like the failed test is a flaky one so not relevant to this change.

@yikf
Copy link
Contributor Author

yikf commented Apr 18, 2023

Yea, I can re-trigger if it's necessary.

@amaliujia
Copy link
Contributor

I guess better to trigger to have it pass

@yikf yikf force-pushed the config-minPartitionNum-doc branch 3 times, most recently from 0ab3ab0 to e5441c3 Compare April 18, 2023 08:44
@HyukjinKwon HyukjinKwon changed the title [MINOR] Improve spark.sql.files.minPartitionNum's doc [MINOR][SQL][DOCS] Improve spark.sql.files.minPartitionNum's doc Apr 18, 2023
@yikf yikf force-pushed the config-minPartitionNum-doc branch 4 times, most recently from 3f192bf to 391ef22 Compare April 20, 2023 03:05
@yikf yikf force-pushed the config-minPartitionNum-doc branch from 391ef22 to a120673 Compare April 21, 2023 08:55
@yikf yikf requested a review from HyukjinKwon April 23, 2023 02:20
@HyukjinKwon
Copy link
Member

Merged to master.

@yikf yikf deleted the config-minPartitionNum-doc branch April 24, 2023 02:10
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.

4 participants