Skip to content
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-32685][SQL][FOLLOW-UP] Update migration guide about change default filed.delim to '\t' when user specifies serde #31051

Closed

Conversation

AngersZhuuuu
Copy link
Contributor

What changes were proposed in this pull request?

Update migration guide according to #30942 (comment)

Why are the changes needed?

update migration guide.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Not need

@AngersZhuuuu AngersZhuuuu changed the title Update sql-migration-guide.md [SPARK-32685][SQL][FOLLOW-UP] Update migration guide about change default filed.delim to '\t' when user specifies serde Jan 6, 2021
@AngersZhuuuu
Copy link
Contributor Author

FYI @cloud-fan @gatorsmile

@SparkQA
Copy link

SparkQA commented Jan 6, 2021

Test build #133705 has finished for PR 31051 at commit b28f4f9.

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

@github-actions github-actions bot added the DOCS label Jan 6, 2021
@SparkQA
Copy link

SparkQA commented Jan 6, 2021

Test build #133715 has finished for PR 31051 at commit 3013ece.

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

@SparkQA
Copy link

SparkQA commented Jan 6, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38303/

@SparkQA
Copy link

SparkQA commented Jan 6, 2021

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38303/

@cloud-fan
Copy link
Contributor

thanks, merging to master!

@cloud-fan cloud-fan closed this in 3cdc4ef Jan 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
4 participants