-
Notifications
You must be signed in to change notification settings - Fork 9.2k
HADOOP-19059. Update AWS SDK to v2.23.7 #6506
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. |
|
This unit test failure is not related to this change: |
This PR only updates the AWS SDK, there is no new feature or other code logic change. There is no need for new tests. |
993b917 to
c709d56
Compare
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
Integration Tests are failing right now, and if you don't report them we will know you aren't running them. No tests: no review. Sorry. testing.md covers SDK qualification. Please do this, again report the results. See #6467 for an example. Ideally cover as much as the s3 test matrix as you can: s3, s3 express, private link, google gcs (yes! really!) and third party. |
|
sorry, I did not post the test report last time, and missed csv test as well. |
Will retry for the tests, it seems many failures this time. |
|
#6515 is covering the failures; the public landsat-pds bucket we were using for tests has been cut. just give us the list of test failures and we will know which to discount. thanks |
|
As per #6507 (comment), I'm marking this PR as deprecated and will be opening a new one with a larger AWS Java SDK upgrade. |
|
this has been obsoleted by #6538 which puts it up to 2.23.19, though it looks like there are more to come. Can I close it? |
|
@steveloughran please go ahead and close (I would, but GH doesn't allow me to) |
Description of PR
Update to AWS SDK v2.23.7 for S3 Access Grants(https://aws.amazon.com/s3/features/access-grants/) support
How was this patch tested?
Run all integration tests with scale, assume role and KMS.
For code changes:
LICENSE,LICENSE-binary,NOTICE-binaryfiles?