-
Notifications
You must be signed in to change notification settings - Fork 910
Adding AWS-CRT-Java 0.4.20 to AWS-SDK-Java-v2 #1607
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
Closed
Closed
Changes from 29 commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
86aba02
Add Support for AWS Common Runtime Http Client
alexw91 08ce630
Fix Race condition in Unit Tests
alexw91 92bc16e
Rebase onto latest master branch
alexw91 0ac39a6
Update AWS CRT Client to use Connection Manager
alexw91 33f9dfc
Update to latest master branch
alexw91 95c24f0
Add aws-crt-client benchmark
andrewhop af8d728
Update CRT Client
alexw91 c697cd5
Update CRT Client and add Stress Test
alexw91 58ac9d5
Benchmark Fixes
alexw91 c94aaaa
Rename to AwsCrtCombinatorialConfigStressIntegrationTest
alexw91 7b45795
Remove CrtResource Dependency injection
alexw91 14f4fc3
Update JavaDocs
alexw91 5b75286
Address Code Review Feedback
alexw91 be3921c
Don't leak Native CrtResources if error occurs during close
alexw91 975af13
Update to new API
alexw91 96bb21a
Update aws-crt-client with up to date parent artifact version
andrewhop a3ba0de
Update to 0.3.35 of the CRT with minor API changes
andrewhop 6cf6e30
Updated crt http client to use latest crt api, which uses byte[] for …
JonathanHenson f4b450b
Removed old comment.
JonathanHenson 1681407
Fixed build failures.
JonathanHenson 24b7801
Fixed benchmark builds.
JonathanHenson 6a72b11
Removed unused import.
JonathanHenson 7ba4045
Fixed SPI test.
JonathanHenson bfab2f3
Hopefully fix the build this time.
JonathanHenson f7a9464
Updating SDK to work with most recent CRT
rccarper d421878
Taking out incorrect try-with-resources block
rccarper 59e5730
Merge branch 'master' into aws-crt-latest-api-update
rccarper 5e85430
Updating to latest SDK snapshot and CRT
rccarper 1570a31
Adjusting usage of builder, removing whitespace change.
rccarper 218d357
Merge branch 'master' into aws-crt-latest-api-update
rccarper a2db25c
Updating snapshot version
rccarper 39e961c
Updating copyright headers
rccarper d6fc0d8
Fixing checkstyle warning for 'TLS' being more than 2 consecutive cap…
rccarper 0cb0e7e
Merge branch 'master' into aws-crt-latest-api-update
rccarper 5f0877e
Updating snapshot and CRT version
rccarper 274b080
Update to next snapshot version: 2.11.7-SNAPSHOT
9d9fd67
Add ellipsis when truncating payload
dagnir 025c43c
Update AWS CRT Client to work with v0.5.1
alexw91 8778f33
Fixed document bean classes used as parameterized types of collection…
bmaizels be3eb1b
DDB Enhanced Client: Added v1/v2 performance comparison tests for all…
bmaizels 69d620a
Merge branch 'master' into aws-crt-latest-api-update
rccarper de2837e
Updating snapshot version and adding aws-crt-client to tests coverage…
rccarper 56a0cd5
Merge branch 'crt_v0.5.1' into aws-crt-latest-api-update
rccarper 0a69d0c
Fixing copyright headers. Fixing CRT version in sdk-benchmarks.
rccarper File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| { | ||
| "category": "AWS Common Runtime Client", | ||
| "type": "bugfix", | ||
| "description": "Upgrade to the latest version (0.3.35) of the AWS Common Runtime." | ||
| } | ||
5 changes: 5 additions & 0 deletions
5
.changes/next-release/feature-AWSCommonRuntimeClient-0306e36.json
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| { | ||
| "category": "AWS Common Runtime Http Client", | ||
| "type": "feature", | ||
| "description": "Add initial support for the AWS Common Runtime (CRT) Http Client." | ||
| } |
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,189 @@ | ||
| <?xml version="1.0" encoding="UTF-8"?> | ||
| <!-- | ||
| ~ Copyright 2010-2019 Amazon.com, Inc. or its affiliates. All Rights Reserved. | ||
| ~ | ||
| ~ Licensed under the Apache License, Version 2.0 (the "License"). | ||
| ~ You may not use this file except in compliance with the License. | ||
| ~ A copy of the License is located at | ||
| ~ | ||
| ~ http://aws.amazon.com/apache2.0 | ||
| ~ | ||
| ~ or in the "license" file accompanying this file. This file is distributed | ||
| ~ on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either | ||
| ~ express or implied. See the License for the specific language governing | ||
| ~ permissions and limitations under the License. | ||
| --> | ||
|
|
||
|
|
||
| <project xmlns="http://maven.apache.org/POM/4.0.0" | ||
| xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" | ||
| xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd"> | ||
| <parent> | ||
| <artifactId>http-clients</artifactId> | ||
| <groupId>software.amazon.awssdk</groupId> | ||
| <version>2.10.57-SNAPSHOT</version> | ||
| </parent> | ||
| <modelVersion>4.0.0</modelVersion> | ||
|
|
||
| <artifactId>aws-crt-client</artifactId> | ||
| <name>AWS Java SDK :: HTTP Clients :: AWS Common Runtime Client</name> | ||
|
|
||
| <dependencies> | ||
| <!--AWS Common Runtime--> | ||
| <dependency> | ||
| <groupId>software.amazon.awssdk.crt</groupId> | ||
| <artifactId>aws-crt</artifactId> | ||
| <version>0.4.20</version> | ||
| </dependency> | ||
|
|
||
| <!--SDK dependencies--> | ||
| <dependency> | ||
| <groupId>software.amazon.awssdk</groupId> | ||
| <artifactId>annotations</artifactId> | ||
| <version>${awsjavasdk.version}</version> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>software.amazon.awssdk</groupId> | ||
| <artifactId>http-client-spi</artifactId> | ||
| <version>${awsjavasdk.version}</version> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>software.amazon.awssdk</groupId> | ||
| <artifactId>utils</artifactId> | ||
| <version>${awsjavasdk.version}</version> | ||
| </dependency> | ||
|
|
||
| <!--Test Dependencies--> | ||
| <dependency> | ||
| <groupId>com.github.tomakehurst</groupId> | ||
| <artifactId>wiremock</artifactId> | ||
| <scope>test</scope> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>org.apache.commons</groupId> | ||
| <artifactId>commons-lang3</artifactId> | ||
| <scope>test</scope> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>junit</groupId> | ||
| <artifactId>junit</artifactId> | ||
| <scope>test</scope> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>org.mockito</groupId> | ||
| <artifactId>mockito-core</artifactId> | ||
| <scope>test</scope> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>org.assertj</groupId> | ||
| <artifactId>assertj-core</artifactId> | ||
| <scope>test</scope> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>org.reactivestreams</groupId> | ||
| <artifactId>reactive-streams-tck</artifactId> | ||
| <scope>test</scope> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>org.slf4j</groupId> | ||
| <artifactId>slf4j-log4j12</artifactId> | ||
| <scope>test</scope> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>log4j</groupId> | ||
| <artifactId>log4j</artifactId> | ||
| <scope>test</scope> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>software.amazon.awssdk</groupId> | ||
| <artifactId>http-client-tests</artifactId> | ||
| <version>${awsjavasdk.version}</version> | ||
| <scope>test</scope> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>software.amazon.awssdk</groupId> | ||
| <artifactId>sdk-core</artifactId> | ||
| <version>${awsjavasdk.version}</version> | ||
| <scope>test</scope> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>software.amazon.awssdk</groupId> | ||
| <artifactId>regions</artifactId> | ||
| <version>${awsjavasdk.version}</version> | ||
| <scope>test</scope> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>software.amazon.awssdk</groupId> | ||
| <artifactId>s3</artifactId> | ||
| <version>${awsjavasdk.version}</version> | ||
| <scope>test</scope> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>software.amazon.awssdk</groupId> | ||
| <artifactId>kms</artifactId> | ||
| <version>${awsjavasdk.version}</version> | ||
| <scope>test</scope> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>software.amazon.awssdk</groupId> | ||
| <artifactId>auth</artifactId> | ||
| <version>${awsjavasdk.version}</version> | ||
| <scope>test</scope> | ||
| </dependency> | ||
| <dependency> | ||
| <artifactId>service-test-utils</artifactId> | ||
| <groupId>software.amazon.awssdk</groupId> | ||
| <version>${awsjavasdk.version}</version> | ||
| <scope>test</scope> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>commons-codec</groupId> | ||
| <artifactId>commons-codec</artifactId> | ||
| <version>${commons-codec.verion}</version> | ||
| <scope>test</scope> | ||
| </dependency> | ||
| </dependencies> | ||
|
|
||
| <build> | ||
| <plugins> | ||
| <!-- The Reactive Streams TCK tests are based on TestNG. See http://maven.apache.org/surefire/maven-surefire-plugin/examples/testng.html#Running_TestNG_and_JUnit_Tests --> | ||
| <plugin> | ||
| <groupId>org.apache.maven.plugins</groupId> | ||
| <artifactId>maven-surefire-plugin</artifactId> | ||
| <version>${maven.surefire.version}</version> | ||
| <configuration> | ||
| <properties> | ||
| <property> | ||
| <name>junit</name> | ||
| <value>false</value> | ||
| </property> | ||
| </properties> | ||
| <threadCount>1</threadCount> | ||
| </configuration> | ||
| <dependencies> | ||
| <dependency> | ||
| <groupId>org.apache.maven.surefire</groupId> | ||
| <artifactId>surefire-junit47</artifactId> | ||
| <version>${maven.surefire.version}</version> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>org.apache.maven.surefire</groupId> | ||
| <artifactId>surefire-testng</artifactId> | ||
| <version>${maven.surefire.version}</version> | ||
| </dependency> | ||
| </dependencies> | ||
| </plugin> | ||
| <plugin> | ||
| <groupId>org.apache.maven.plugins</groupId> | ||
| <artifactId>maven-jar-plugin</artifactId> | ||
| <configuration> | ||
| <archive> | ||
| <manifestEntries> | ||
| <Automatic-Module-Name>software.amazon.awssdk.http.crt</Automatic-Module-Name> | ||
| </manifestEntries> | ||
| </archive> | ||
| </configuration> | ||
| </plugin> | ||
| </plugins> | ||
| </build> | ||
| </project> |
Oops, something went wrong.
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.
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.
I don't think we need this in here since this PR is the initial release of the CRT in SDK
Uh oh!
There was an error while loading. Please reload this page.
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.
Removed in new PR