Skip to content

[SQL][DOC] Fix a default name for parquet compression #14351

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.

Sign up for GitHub

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

Jump to bottom
Closed
maropu wants to merge 1 commit into from
Closed

[SQL][DOC] Fix a default name for parquet compression #14351

maropu wants to merge 1 commit into from

Conversation

@maropu
Copy link
Member

@maropu maropu commented Jul 25, 2016

What changes were proposed in this pull request?

This pr is to fix a wrong description for parquet default compression.

@maropu
Copy link
Member Author

maropu commented Jul 25, 2016

@rxin could you check this before the v2.0 release?
An actual default seems to be snappy: https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala#L215

@rxin
Copy link
Contributor

rxin commented Jul 25, 2016

Thanks - merging in master/2.0.

@asfgit asfgit closed this in cda4603 Jul 25, 2016
asfgit pushed a commit that referenced this pull request Jul 25, 2016
@maropu @rxin
[SQL][DOC] Fix a default name for parquet compression
1b4f7cf 
## What changes were proposed in this pull request?
This pr is to fix a wrong description for parquet default compression.

Author: Takeshi YAMAMURO <linguin.m.s@gmail.com>

Closes #14351 from maropu/FixParquetDoc.

(cherry picked from commit cda4603)
Signed-off-by: Reynold Xin <rxin@databricks.com>
@SparkQA
Copy link

SparkQA commented Jul 25, 2016

Test build #62844 has finished for PR 14351 at commit 4bff124.

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

@maropu maropu deleted the FixParquetDoc branch July 5, 2017 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@maropu @rxin @SparkQA