Looks fine to me Daniel.
Thanks, Michael. On 06/06/2017, 16:48, Daniel Fuchs wrote:
Hi, Please find below a patch that fixes several race conditions observed in the jdk.incubator.httpclient module. In particular this fixes: 8180044: java/net/httpclient/ManyRequests.java failed due to timeout https://bugs.openjdk.java.net/browse/JDK-8180044 and 8181430: HTTP/2 client might deadlock when receiving data during the initial handshake https://bugs.openjdk.java.net/browse/JDK-8181430 webrev: http://cr.openjdk.java.net/~dfuchs/webrev_8180044%2b8181430/webrev.00/ Testing: I have run the tests in a loop all week end. More info on the fix:https://bugs.openjdk.java.net/browse/JDK-8180044?focusedCommentId=14090708&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14090708 https://bugs.openjdk.java.net/browse/JDK-8181430?focusedCommentId=14089704&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14089704 https://bugs.openjdk.java.net/browse/JDK-8181430?focusedCommentId=14090734&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14090734best regards, -- daniel
