Add CRT HTTP/2 stability tests and JMH benchmarks#6723
Add CRT HTTP/2 stability tests and JMH benchmarks#6723zoewangg wants to merge 3 commits intofeature/master/crtH2from
Conversation
zoewangg
force-pushed
the
zoewang/addPerfTestsStabilityTests
branch
2 times, most recently
from
1e59327 to
11cfe9c
Compare
# Conflicts: # services/kinesis/src/it/java/software/amazon/awssdk/services/kinesis/AbstractTestCase.java
zoewangg
force-pushed
the
zoewang/addPerfTestsStabilityTests
branch
from
1d38de5 to
05f773d
Compare
| assertThatThrownBy(() -> recorder.completeFuture().get(5, TimeUnit.SECONDS)).hasCauseInstanceOf(IOException.class) | ||
| .hasMessageContaining("socket"); |
There was a problem hiding this comment.
Updated this test because it's a bit flaky, when I run it locally, it fails of socket timeout but when the CI runs it, it sometimes fails of operation timeout
| .hasMessageContaining("operation timed out"); | ||
| .hasMessageContaining("socket"); |
There was a problem hiding this comment.
Same here. Updated this test because it's a bit flaky, when I run it locally, it fails of socket timeout but when the CI runs it, it sometimes fails of operation timeout
There was a problem hiding this comment.
if it's flaky wouldn't we want to assert that it has either one of those strings?
|
| import software.amazon.awssdk.http.Protocol; | ||
| import software.amazon.awssdk.http.SdkHttpConfigurationOption; |
| .hasMessageContaining("operation timed out"); | ||
| .hasMessageContaining("socket"); |
There was a problem hiding this comment.
if it's flaky wouldn't we want to assert that it has either one of those strings?
2 participants
Motivation and Context
This PR adds stability tests and JMH benchmarks for the CRT HTTP client with HTTP/2 protocol support. These tests validate CRT HTTP/2 behavior under high-concurrency
scenarios using real AWS services (Kinesis and Transcribe Streaming) that support HTTP/2.
Modifications
New Stability Tests:
New JMH Benchmark:
Refactoring:
Benchmark Results (CRT HTTP/2 vs Netty HTTP/2):
Testing
Types of changes
Checklist