Skip to content

Conversation

@dcoliversun
Copy link
Contributor

What changes were proposed in this pull request?

This PR aims to supplement undocumented orc configurations in documentation.

Why are the changes needed?

Help users to confirm configurations through documentation instead of code.

Does this PR introduce any user-facing change?

Yes, more configurations in documentations.

How was this patch tested?

Pass the GA.

@dcoliversun
Copy link
Contributor Author

cc @dongjoon-hyun @srowen
It would be good if you have a time to review

@github-actions github-actions bot added the DOCS label Oct 10, 2022
@srowen srowen closed this in 9e8198d Oct 10, 2022
@srowen
Copy link
Member

srowen commented Oct 10, 2022

Merged to master

@dcoliversun dcoliversun deleted the SPARK-40726 branch October 11, 2022 01:28
@dcoliversun
Copy link
Contributor Author

Thanks for @srowen review

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, late LGTM. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants