Skip to content

Commit d8ab842

Browse files
HADOOP-18068. upgrade AWS SDK to 1.12.132 (apache#3864)
With this update, the versions of key shaded dependencies are jackson 2.12.3 httpclient 4.5.13 Contributed by Steve Loughran
1 parent 14ba19a commit d8ab842

File tree

4 files changed

+18
-6
lines changed

4 files changed

+18
-6
lines changed

NOTICE-binary

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ available from http://www.digip.org/jansson/.
6666

6767

6868
AWS SDK for Java
69-
Copyright 2010-2014 Amazon.com, Inc. or its affiliates. All Rights Reserved.
69+
Copyright 2010-2022 Amazon.com, Inc. or its affiliates. All Rights Reserved.
7070

7171
This product includes software developed by
7272
Amazon Technologies, Inc (http://www.amazon.com/).

hadoop-project/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,7 @@
185185
<exec-maven-plugin.version>1.3.1</exec-maven-plugin.version>
186186
<make-maven-plugin.version>1.0-beta-1</make-maven-plugin.version>
187187
<surefire.fork.timeout>900</surefire.fork.timeout>
188-
<aws-java-sdk.version>1.11.1026</aws-java-sdk.version>
188+
<aws-java-sdk.version>1.12.132</aws-java-sdk.version>
189189
<hsqldb.version>2.3.4</hsqldb.version>
190190
<frontend-maven-plugin.version>1.11.2</frontend-maven-plugin.version>
191191
<jasmine-maven-plugin.version>2.1</jasmine-maven-plugin.version>

hadoop-tools/hadoop-aws/src/site/markdown/tools/hadoop-aws/testing.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1249,6 +1249,7 @@ to AWS services.
12491249
* Try and get other people, especially anyone with their own endpoints,
12501250
apps or different deployment environments, to run their own tests.
12511251
* Run the load tests, especially `ILoadTestS3ABulkDeleteThrottling`.
1252+
* Checkout cloudstore, build it against your version of hadoop, then use its CLI to run some commands (`storediag` etc)
12521253
12531254
### Dealing with Deprecated APIs and New Features
12541255

hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/TestArnResource.java

Lines changed: 15 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919
package org.apache.hadoop.fs.s3a;
2020

2121
import com.amazonaws.regions.Regions;
22+
import org.assertj.core.api.Assertions;
2223
import org.junit.Test;
2324
import org.slf4j.Logger;
2425
import org.slf4j.LoggerFactory;
@@ -27,6 +28,14 @@
2728

2829
import static org.apache.hadoop.test.LambdaTestUtils.intercept;
2930

31+
/**
32+
* Verifies the mapping of ARN declaration of resource to the associated
33+
* access point.
34+
* The region mapping assertions have been brittle to changes across AWS SDK
35+
* versions to only verify partial matches, rather than the FQDN of the
36+
* endpoints.
37+
*
38+
*/
3039
public class TestArnResource extends HadoopTestBase {
3140
private final static Logger LOG = LoggerFactory.getLogger(TestArnResource.class);
3241

@@ -37,10 +46,10 @@ public void parseAccessPointFromArn() throws IllegalArgumentException {
3746
String accessPoint = "testAp";
3847
String accountId = "123456789101";
3948
String[][] regionPartitionEndpoints = new String[][] {
40-
{Regions.EU_WEST_1.getName(), "aws", "s3-accesspoint.eu-west-1.amazonaws.com"},
49+
{Regions.EU_WEST_1.getName(), "aws", "eu-west-1.amazonaws.com"},
4150
{Regions.US_GOV_EAST_1.getName(), "aws-us-gov",
42-
"s3-accesspoint.us-gov-east-1.amazonaws.com"},
43-
{Regions.CN_NORTH_1.getName(), "aws-cn", "s3-accesspoint.cn-north-1.amazonaws.com.cn"},
51+
"us-gov-east-1.amazonaws.com"},
52+
{Regions.CN_NORTH_1.getName(), "aws-cn", "cn-north-1.amazonaws.com"},
4453
};
4554

4655
for (String[] testPair : regionPartitionEndpoints) {
@@ -57,7 +66,9 @@ public void parseAccessPointFromArn() throws IllegalArgumentException {
5766
assertEquals("Access Point name does not match", accessPoint, resource.getName());
5867
assertEquals("Account Id does not match", accountId, resource.getOwnerAccountId());
5968
assertEquals("Region does not match", region, resource.getRegion());
60-
assertEquals("Endpoint does not match", endpoint, resource.getEndpoint());
69+
Assertions.assertThat(resource.getEndpoint())
70+
.describedAs("Endpoint does not match")
71+
.contains(endpoint);
6172
}
6273
}
6374

0 commit comments

Comments
 (0)