New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SPARK-39035][SQL][TESTS] Add tests for options from to_csv
and from_csv
.
#36401
Conversation
There was a problem hiding this 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. cc @tgravescs FYI
Merged to master. |
Sorry, the tests failed https://github.com/apache/spark/runs/6226791783 when ANSI mode ( @itholic can you run the tests with this mode, and make it pass regardless of this configuration? |
I reverted this. |
Approval dismissed since this got reverted.
to_csv
and from_csv
.to_csv
and from_csv
.
Merged to master. |
What changes were proposed in this pull request?
This PR proposes to fill up the missing test for options from CSV read/write functions.
Why are the changes needed?
For better testability and correct the documents.
Does this PR introduce any user-facing change?
No, it's dev-only.
How was this patch tested?
Added some UTs for CSV read/write options.