-
Notifications
You must be signed in to change notification settings - Fork 9.1k
HADOOP-19224 Upgrade esdk to the latest version 3.24.3 #7003
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
Conversation
💔 -1 overall
This message was automatically generated. |
you've got a couple of checkbox items there
If you've not submitted a PR which goes near any of the stores, know that as our CI tooling isn't trusted with credentials, submitters are required to do this before anyone will review the patch. Nothing personal: we are that ruthless with each other too. |
We're closing this stale PR because it has been open for 100 days with no activity. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable. |
reopening this I do want the update; HADOOP-19696 adds the option to include esdk into the redistributables. It'll be off by default as we don't want to okhttp in there again as it's a pain. @melin did you run the tests against the store? |
💔 -1 overall
This message was automatically generated. |
Description of PR
How was this patch tested?
For code changes:
LICENSE
,LICENSE-binary
,NOTICE-binary
files?