ok2c commented on PR #709:
URL: 
https://github.com/apache/httpcomponents-client/pull/709#issuecomment-3229349684

   > I'm not sure what you mean. 
   
   @rschmitt Okay. I will have to be blunt, then
   
   > The tests are all doing what I intend them to do.
   
   Then, in their present form the tests are meaningless / useless for HTTP/1.1 
due to the request pipelining. There is no point committing them in their 
present shape
   
   Some do 
   
   ```
                   final Future<SimpleHttpResponse> f = sendPing(client);
                   try {
                       f.get();
                   } catch (final ExecutionException ignore) {
                   }
                   futures.add(f);
   ```
   which is correct.
   
   Others do 
   ```
                   futures.add(sendPing(client));
   ```
   which is wrong in the given context due to the use of request pipleining 
with the HTTP/1.1 transport.
   
   > I bet if you [grab the latest 
version](https://docs.aws.amazon.com/corretto/latest/corretto-8-ug/downloads-list.html)
 you'll be able to reproduce the problem.
   
   I will get the latest Java 8 JRE and will try out to reproduce the defect 
with it.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to