HADOOP-18344. Upgrade AWS SDK to 1.12.262 #4637
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Test failures
I will need to see if this is a regression. I believe i have disabled setting storage class. |
|
CLI tests good. but they are all logging a warning about unset storage class. Filed HADOOP-18371 |
|
setup failure in ILoadTestS3ABulkDeleteThrottling; filed HADOOP-18372 |
|
test failures occur on trunk even without sdk upgrade |
|
ILoadTestS3ABulkDeleteThrottling (also failing without the upgrade) is fixed in #4642 |
|
the storage class tests all work when i switch from bytebuffer as the upload buffering, as does the OOM. |
|
CSE-KMS mode failing, but as i've not tried it before, can't blame the SDK SSE-KMS works so it could be key config/permissions. |
Fixes CVE-2018-7489 in shaded jackson. +Add more commands in testing.md to the CLI tests needed when qualifying a release Change-Id: If8020ad581d290f0cbe322184e860b1e3f4aeffe
steveloughran
force-pushed
the
s3/HADOOP-18344-AWS-SDK-1.12.262
branch
from
9800dd2 to
7e1b620
Compare
This was referenced Jul 27, 2022
|
💔 -1 overall
This message was automatically generated. |
remove a space at an EOL Change-Id: I819e210f988b58281440d6834aea4203e20ccee1
|
Okay looks good. running the tests once complete will give a +1 |
|
💔 -1 overall
This message was automatically generated. |
mukund-thakur
approved these changes
Jul 27, 2022
| # expect the iostatistics object_list_request value to be O(directories) | ||
| bin/hadoop fs -ls -R $BUCKET/ | ||
|
|
||
| # expect the iostatistics object_list_request and op_get_content_summary values to be 1 |
There was a problem hiding this comment.
Yes there is a extra space here at the end as pointed by Yetus,
There was a problem hiding this comment.
fixed that last night; will push and then merge
asfgit
pushed a commit
that referenced
this pull request
Jul 28, 2022
Update LICENSE-binary with the new AWS SDK version. Followup to #4637. Contributed by Steve Loughran
asfgit
pushed a commit
that referenced
this pull request
Jul 28, 2022
Fixes CVE-2018-7489 in shaded jackson. +Add more commands in testing.md to the CLI tests needed when qualifying a release Contributed by Steve Loughran
asfgit
pushed a commit
that referenced
this pull request
Jul 28, 2022
Fixes CVE-2018-7489 in shaded jackson. +Add more commands in testing.md to the CLI tests needed when qualifying a release Contributed by Steve Loughran
|
aah, forgot that license-binary change. pushed up a followup, and for the backports merged the two changes |
|
💔 -1 overall
This message was automatically generated. |
HarshitGupta11
pushed a commit
to HarshitGupta11/hadoop
that referenced
this pull request
Nov 28, 2022
Fixes CVE-2018-7489 in shaded jackson. +Add more commands in testing.md to the CLI tests needed when qualifying a release Contributed by Steve Loughran
HarshitGupta11
pushed a commit
to HarshitGupta11/hadoop
that referenced
this pull request
Nov 28, 2022
Update LICENSE-binary with the new AWS SDK version. Followup to apache#4637. Contributed by Steve Loughran
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of PR
How was this patch tested?
object store test in progress. first run had two failures I'd not see before