You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@thrift.apache.org by GitBox <gi...@apache.org> on 2020/10/07 19:05:20 UTC

[GitHub] [thrift] dcelasun commented on pull request #2256: THRIFT-5240: Disable go server side connectivity check by default

dcelasun commented on pull request #2256:
URL: https://github.com/apache/thrift/pull/2256#issuecomment-705135142


   I'm not so sure about a 5ms default. We have a fairly loaded Go 1.15 server (so with the ticker optimization) with 5ms/8ms p50/p75 latency respectively and a fixed 5ms increase seems a bit too much.
   
   Would there be enough cpu load reduction with a 2-3ms ticker?


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org