rschmitt commented on PR #606:
URL: 
https://github.com/apache/httpcomponents-core/pull/606#issuecomment-3826134485

   > Once available in client `H2PingPolicy` should go into `ConnectionConfig`
   
   I think we have too many config knobs already. I'd really prefer to expose 
this through `setValidateAfterInactivity`. Once we have support for HTTP/2 
keep-alive PINGs that run asynchronously in the background, no one in their 
right mind will ever want to send a blocking PING in the request path to 
validate a connection. (I've spent this week dealing with a set of production 
issues that are directly related to this problem, and it has radicalized me re: 
how badly I want this keep-alive `PING` functionality.)


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