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-42488][BUILD] Upgrade commons-crypto to 1.2.0 #40082
Conversation
test first |
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.
+1, LGTM. Thank you, @LuciferYang and @srowen .
Merged to master.
Thanks @dongjoon-hyun @srowen |
BTW, I have one question, @LuciferYang . We explicitly exclude Lines 2679 to 2689 in 5fc44da
|
From the release notes, I think nothing is losing.
|
Oh, this seems to break Apple Silicon environment, @LuciferYang . I hit the failure on Macbook environment.
|
In short, it's
|
Thanks @dongjoon-hyun, it's my bad, I hit same issue, let me revert this one, I will remember check this part next time. |
Well, let me revert directly~ |
Since it's on master only, it's easier. |
ok, thanks @dongjoon-hyun |
It's reverted via ca22c41 . |
FYI, I also confimed that the revert commit recovers the following streaming module test, too.
|
What changes were proposed in this pull request?
This pr aims upgrade commons-crypto from 1.1.0 to 1.2.0.
Why are the changes needed?
The new features in 1.2.0 as follows:
https://github.com/apache/commons-crypto/blob/1ebfddd0e77585884872416a0dff2dd114a88864/RELEASE-NOTES.txt#L12-L21
Does this PR introduce any user-facing change?
No
How was this patch tested?
Pass GitHub Actions