Skip to content

[SPARK-27267][CORE] Update snappy to avoid error when decompressing empty serialized data #531

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
Merged
merged 1 commit into from
Apr 11, 2019
Merged

[SPARK-27267][CORE] Update snappy to avoid error when decompressing empty serialized data #531

merged 1 commit into from
Apr 11, 2019

Conversation

@ntrinquier
Copy link

@ntrinquier ntrinquier commented Apr 5, 2019 •
edited
Loading

@srowen
[SPARK-27267][CORE] Update snappy to avoid error when decompressing e…
144fc7e 
…mpty serialized data

(See JIRA for problem statement)

Update snappy 1.1.7.1 -> 1.1.7.3 to pick up an empty-stream and Java 9 fix.

There appear to be no other changes of consequence:
https://github.com/xerial/snappy-java/blob/master/Milestone.md

Existing tests

Closes apache#24242 from srowen/SPARK-27267.

Authored-by: Sean Owen <[email protected]>
Signed-off-by: Sean Owen <[email protected]>
@robert3005
Copy link

robert3005 commented Apr 11, 2019

👍

@bulldozer-bot bulldozer-bot bot merged commit f2b22a1 into palantir:master Apr 11, 2019
@ntrinquier ntrinquier deleted the nt/SPARK-27267 branch April 11, 2019 14:43
@vinooganesh vinooganesh mentioned this pull request Apr 19, 2019
Merged
bulldozer-bot bot pushed a commit that referenced this pull request Apr 19, 2019
@vinooganesh @bulldozer-bot
Pull commits in to release 2.5.0-palantir.X (#539)
9a28591 
* [SPARK-27267][CORE] Update snappy to avoid error when decompressing empty serialized data (#531)
* [SPARK-27514][SQL] Skip collapsing windows with empty window expressions (#538)
* Bump hadoop to 2.9.2-palantir.5 (#537)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@gatesn gatesn gatesn approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ntrinquier @robert3005 @gatesn @srowen