-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
PARQUET-1967: Upgrade Zstd-jni to 1.4.8-3 #859
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. There are a couple unrelated Travis failures for our PRs. Let's wait for a green build.
Thank you for reviews. |
Do I need to rebase this PR to see the green build?
|
@dongjoon-hyun, you don't need to rebase, I've already re-triggered the build. I don't know what's happening with Travis but there are unrelated failures and stuck builds in the system for several PRs. I hope, it'll be solved soon. |
Got it. Thank you, @gszadovszky ! |
Close/re-open this PR to trigger the newly configured github action. |
Thank you for retriggering, @gszadovszky ! |
Thank you so much! |
Thank you for the contribution, @dongjoon-hyun! :) |
Make sure you have checked all steps below.
Jira
Tests
This PR should pass the existing UTs and passed all UTs.
Commits
Documentation