Skip to content
This repository was archived by the owner on Jun 19, 2026. It is now read-only.

Support to specify a shorten name for compression codec - #235

Closed
HyukjinKwon wants to merge 10 commits into
databricks:masterfrom
HyukjinKwon:ISSUE-234-shorten-name
Closed

Support to specify a shorten name for compression codec#235
HyukjinKwon wants to merge 10 commits into
databricks:masterfrom
HyukjinKwon:ISSUE-234-shorten-name

Conversation

@HyukjinKwon

@HyukjinKwonHyukjinKwon commented Jan 20, 2016

Copy link
Copy Markdown
Member

#234

This PR is similar with apache/spark#10805.

This PR adds the support for shorten names for compression codecs and added a CompressionCodecs class instead of the implicit function as its use is nor recommended.

@codecov-io

codecov-io commented Jan 20, 2016

Copy link
Copy Markdown

Current coverage is 86.07%

Merging #235 into master will increase coverage by +0.02% as of 5761d56

@@ master #235 diff @@
======================================
Files 11 12 +1
Stmts 509 517 +8
Branches 148 149 +1
Methods 0 0 ======================================
+ Hit 438 445 +7
Partial 0 0 - Missed 71 72 +1

Review entire Coverage Diff as of 5761d56

Powered by Codecov. Updated on successful CI builds.

@falaki

falaki commented Jan 20, 2016

Copy link
Copy Markdown
Member

@HyukjinKwon thanks for submitting this. Would you mind adding a small unit test for CompressionCodecs?

@HyukjinKwon

HyukjinKwon commented Jan 21, 2016

Copy link
Copy Markdown
MemberAuthor

@falaki Sure.

@HyukjinKwonHyukjinKwon mentioned this pull request Feb 1, 2016
Closed
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

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

@HyukjinKwon@codecov-io@falaki