Re: RFR 8178699/10, Fail to send async requests if server doesn't response the first one

2017-06-23 Thread Felix Yang
Hi Daniel, thread dump can be found in JBS attachment . Thanks -Felix Sent from my iphone > 在 2017年6月23日,18:40,Daniel Fuchs 写道: > > Hi Felix, > > Looks good to me. > > +1 for removing unnecessary calls that need synchronization and > cause deadlocks. > > If you still have the full thr

Re: RFR 8178699/10, Fail to send async requests if server doesn't response the first one

2017-06-23 Thread Daniel Fuchs
Hi Felix, Looks good to me. +1 for removing unnecessary calls that need synchronization and cause deadlocks. If you still have the full thread-dump where the issue could be observed it might be good to record it in JDK-8178699 (if you no longer have it then don't bother). best regards, -- dan

RFR 8178699/10, Fail to send async requests if server doesn't response the first one

2017-06-23 Thread Felix Yang
Hi all, please review the following patch to API in jdk.incubator.httpclient. The patch is more like a clean-up: * With current RequestProcessor implementation, constructing multiple HTTP Exchange may be blocked if the first request doesn't response by server side. This is introduced