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-33725][BUILD][3.1] Upgrade snappy-java to 1.1.8.2 #30695

Closed
wants to merge 1 commit into from

Conversation

viirya
Copy link
Member

@viirya viirya commented Dec 9, 2020

What changes were proposed in this pull request?

This upgrades snappy-java to 1.1.8.2.

Why are the changes needed?

Minor version upgrade that includes:

  • Fixed an initialization issue when using a recent Mac OS X version
  • Support Apple Silicon (M1, Mac-aarch64)
  • Fixed the pure-java Snappy fallback logic when no native library for your platform is found.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Unit test.

@viirya
Copy link
Member Author

viirya commented Dec 9, 2020

cc @dongjoon-hyun

@github-actions github-actions bot added the BUILD label Dec 9, 2020
@dongjoon-hyun
Copy link
Member

Thanks!

@dongjoon-hyun
Copy link
Member

dongjoon-hyun commented Dec 9, 2020

BTW, could you make backporting PRs to branch-3.0/branch-2.4 seperately? It would be great to see the test result if possible. For those branches, snappy-java is older version. So, it's a little bigger change.

@viirya
Copy link
Member Author

viirya commented Dec 9, 2020

BTW, could you make backporting PRs to branch-3.0/branch-2.4 seperately? It would be great to see the test result if possible. For those branches, snappy-java is older version. So, it's a little bigger change.

Okay. yea, these branches still use 1.1.7. Let's see the test result.

@SparkQA
Copy link

SparkQA commented Dec 10, 2020

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

@SparkQA
Copy link

SparkQA commented Dec 10, 2020

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

@SparkQA
Copy link

SparkQA commented Dec 10, 2020

Test build #132521 has finished for PR 30695 at commit 77559b9.

  • This patch fails SparkR unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM. Thank you all!
Merged to branch-3.1.

dongjoon-hyun pushed a commit that referenced this pull request Dec 10, 2020
### What changes were proposed in this pull request?

This upgrades snappy-java to 1.1.8.2.

### Why are the changes needed?

Minor version upgrade that includes:

- [Fixed](xerial/snappy-java#265) an initialization issue when using a recent Mac OS X version
- Support Apple Silicon (M1, Mac-aarch64)
- Fixed the pure-java Snappy fallback logic when no native library for your platform is found.

### Does this PR introduce _any_ user-facing change?

No

### How was this patch tested?

Unit test.

Closes #30695 from viirya/upgrade-snappy-3.1.

Authored-by: Liang-Chi Hsieh <viirya@gmail.com>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
@viirya viirya deleted the upgrade-snappy-3.1 branch December 27, 2023 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
5 participants