Skip to content

AWS Common Runtime HTTP client preview #2021

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 129 commits into from
Sep 5, 2020
Merged

AWS Common Runtime HTTP client preview #2021

merged 129 commits into from
Sep 5, 2020

Conversation

zoewangg
Copy link
Contributor

@zoewangg zoewangg commented Sep 4, 2020 •
edited
Loading

Description

  • Merge aws-crt-dev-preview in master
  • Update aws-crt-client version to be PREVIEW in cd9ed9a and 4c13fad

Screenshots (if appropriate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist

  • I have read the CONTRIBUTING document
  • Local run of mvn install succeeds
  • My code follows the code style of this project
  • My change requires a change to the Javadoc documentation
  • I have updated the Javadoc documentation accordingly
  • I have read the README document
  • I have added tests to cover my changes
  • All new and existing tests passed
  • A short description of the change has been added to the CHANGELOG
  • My change is to implement 1.11 parity feature and I have updated LaunchChangelog

License

  • I confirm that this pull request can be released under the Apache 2 license

alexw91 and others added 30 commits September 9, 2019 10:00
@alexw91 @millems
Add Support for AWS Common Runtime Http Client
37070b1
@alexw91 @millems
Fix Race condition in Unit Tests
e4e05de
@alexw91 @millems
Rebase onto latest master branch
3af0d5d
@alexw91 @millems
Update AWS CRT Client to use Connection Manager
d9ad52c
@alexw91 @millems
Update to latest master branch
b48b6f8
@andrewhop @millems
Add aws-crt-client benchmark
16e7e5a
@alexw91 @millems
Update CRT Client
b57d1ed
@alexw91 @millems
Update CRT Client and add Stress Test
e041e57
@alexw91 @millems
Benchmark Fixes
eb06fd2
@alexw91 @millems
Rename to AwsCrtCombinatorialConfigStressIntegrationTest
1e02f56
@alexw91 @millems
Remove CrtResource Dependency injection
87d92b0
@alexw91 @millems
Update JavaDocs
7d02a4c
@alexw91 @millems
Address Code Review Feedback
326c0e0
@alexw91 @millems
Don't leak Native CrtResources if error occurs during close
0878ffb
@alexw91 @millems
Update to new API
c159d0c
@andrewhop
Merge branch 'master' into update
060092f
@andrewhop
Update aws-crt-client with up to date parent artifact version
08e4c52
@dagnir
Merge pull request #1495 from andrewhop/update
d070541 
Merge in latest changes from master to aws-crt-dev-preivew
@andrewhop
Update to 0.3.35 of the CRT with minor API changes
302c149
@dagnir
Merge pull request #1485 from andrewhop/aws-crt-dev-preview
efb7257 
Update to 0.3.35 of the CRT with minor API changes
@alexw91 @rccarper
Add Support for AWS Common Runtime Http Client
eede205
@alexw91 @rccarper
Fix Race condition in Unit Tests
c5af239
@alexw91 @rccarper
Rebase onto latest master branch
8d5feb9
@alexw91 @rccarper
Update AWS CRT Client to use Connection Manager
0bae5db
@alexw91 @rccarper
Update to latest master branch
10eab3e
@andrewhop @rccarper
Add aws-crt-client benchmark
fbce50d
@alexw91 @rccarper
Update CRT Client
5429e87
@alexw91 @rccarper
Update CRT Client and add Stress Test
0833b3c
@alexw91 @rccarper
Benchmark Fixes
78f708e
@alexw91 @rccarper
Rename to AwsCrtCombinatorialConfigStressIntegrationTest
f27da8c
bretambrose and others added 21 commits July 15, 2020 14:32
@bretambrose
Doing things correctly is hard
320b132
@bretambrose
Missing dependency
8415cfc
@zoewangg
Merge pull request #1777 from bretambrose/aws-crt-dev-preview
28de055 
Aws crt dev preview
@bretambrose
Update to latest crt with new windowing implementation
db93b7d
@zoewangg
Merge pull request #1964 from bretambrose/aws-crt-dev-preview
21dab69 
Update CRT http client to latest published CRT
@zoewangg
Merge remote-tracking branch 'origin' into aws-crt-dev-preview
59757e4
@zoewangg
Update version
0a249ae
@zoewangg
Various refactoring of crt branch
5ecfb2d
@zoewangg
update stability tests
1f0942c
@zoewangg
Various updates: renaming methods, adding create method, adding AwsCr…
5e0266b 
…tSdkHttpService
@zoewangg
Create ConnectionHealthChecksConfiguration
feebcd2
@zoewangg
Refactoring - moving the request exceution logic to a separate class …
9ba3d85 
…and add tests
@zoewangg
Invoke responseHandler#onError when the request future is cancelled
82bb676
@zoewangg
Bump up crt version adn remove elg and host resolver configs
8bed1b8
@zoewangg
Add preview API annotations
ab747a5
@zoewangg
Merge branch 'master' into aws-crt-dev-preview
69d0d60
@zoewangg
Add preview version
cd9ed9a
@zoewangg
Merge remote-tracking branch 'origin/master' into zoewang/crtUpdates-…
4d7f667 
…previewVersion
@zoewangg
Update version
4c13fad
@zoewangg
Merge remote-tracking branch 'origin/master' into zoewang/crtUpdates-…
4df5638 
…previewVersion
@zoewangg
Update snapshot version
bc4adf7
@zoewangg
Copy link
Contributor Author

zoewangg commented Sep 4, 2020

Stability & integ tests passed.

@zoewangg zoewangg force-pushed the zoewang/crtUpdates-previewVersion branch from 51dc0a2 to e3892f0 Compare September 4, 2020 23:09
bmaizels
bmaizels approved these changes Sep 4, 2020
View reviewed changes
Copy link
Contributor

@bmaizels bmaizels left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the change log and the pom.xml. Both look good.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 4, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 23 Code Smells

82.5% 82.5% Coverage
2.0% 2.0% Duplication

warning The version of Java (1.8.0_252) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11.
Read more here

@zoewangg zoewangg merged commit abee620 into master Sep 5, 2020
@zoewangg zoewangg deleted the zoewang/crtUpdates-previewVersion branch September 8, 2020 20:02
@dagnir dagnir mentioned this pull request Apr 7, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bmaizels bmaizels bmaizels approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@zoewangg @bmaizels @alexw91 @andrewhop @dagnir @rccarper @JonathanHenson @bretambrose