arturobernalg commented on PR #730:
URL: 
https://github.com/apache/httpcomponents-core/pull/730#issuecomment-6569783241

   @ok2c I have added the counter: the accept loop now takes at most 100 
connections per selection event and returns to the selector, which reports 
OP_ACCEPT again if more are pending, so nothing is lost. I checked that with 
600 simultaneous connections against master and against the change: all 600 are 
served in both cases. I also added a test that opens 150 connections, more than 
the limit, and expects all of them to be accepted. It is skipped when the 
system does not have enough file descriptors. The accept failure test is 
unchanged. Everything is in 870c2c3. The limit of 100 is arbitrary, tell me if 
you prefer another value.


-- 
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