stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8289385: Cleanup redundant synchronization in Http2ClientImpl

Browse Source 
Reviewed-by: jpai, dfuchs
This commit is contained in:
Andrey Turbanov 2022-07-02 15:24:23 +00:00
parent cdf6979259
commit dee5121bd4
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/java.net.http/share/classes/jdk/internal/net/http/Http2ClientImpl.java
View File

@ -31,7 +31,6 @@ import java.io.UncheckedIOException;
import java.net.InetSocketAddress;
import java.net.URI;
import java.util.Base64;
import java.util.Collections;
import java.util.HashSet;
import java.util.Map;
import java.util.Set;
@ -57,7 +56,9 @@ class Http2ClientImpl {
Utils.getDebugLogger("Http2ClientImpl"::toString, Utils.DEBUG);
private final HttpClientImpl client;
private volatile boolean stopping;
// only accessed from within synchronized blocks
private boolean stopping;
Http2ClientImpl(HttpClientImpl client) {
this.client = client;
@ -66,7 +67,8 @@ class Http2ClientImpl {
/* Map key is "scheme:host:port" */
private final Map<String,Http2Connection> connections = new ConcurrentHashMap<>();
private final Set<String> failures = Collections.synchronizedSet(new HashSet<>());
// only accessed from within synchronized blocks
private final Set<String> failures = new HashSet<>();
/**
* When HTTP/2 requested only. The following describes the aggregate behavior including the