Skip to content

Commit e20639d

Browse files
John Viegasjoviegas
John Viegas
authored andcommitted
Reverting #1971 as it was causing latency issues in test canaries
1 parent 85c052d commit e20639d

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
{
2+
"category": "AWS SDK for Java v2",
3+
"contributor": "",
4+
"type": "bugfix",
5+
"description": "Reverting synchronization of eventsToDeliver Object in onComplete() [PR#1971](https://github.com/aws/aws-sdk-java-v2/pull/1971) since it was causing latency."
6+
}

core/aws-core/src/main/java/software/amazon/awssdk/awscore/eventstream/EventStreamAsyncResponseTransformer.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -394,9 +394,7 @@ public void onError(Throwable throwable) {
394394
@Override
395395
public void onComplete() {
396396
// Add the special on complete event to signal drainEvents to complete the subscriber
397-
synchronized (eventsToDeliver) {
398-
eventsToDeliver.add(ON_COMPLETE_EVENT);
399-
}
397+
eventsToDeliver.add(ON_COMPLETE_EVENT);
400398
drainEventsIfNotAlready();
401399
transformFuture.complete(null);
402400
}

0 commit comments

Comments
 (0)