kfaraz commented on code in PR #18493:
URL: https://github.com/apache/druid/pull/18493#discussion_r2348140602


##########
docs/configuration/index.md:
##########
@@ -702,7 +702,7 @@ All Druid components can communicate with each other over 
HTTP.
 |`druid.global.http.compressionCodec`|Compression codec to communicate with 
others. May be "gzip" or "identity".|`gzip`|
 |`druid.global.http.readTimeout`|The timeout for data reads.|`PT15M`|
 |`druid.global.http.unusedConnectionTimeout`|The timeout for idle connections 
in connection pool. The connection in the pool will be closed after this 
timeout and a new one will be established. This timeout should be less than 
`druid.global.http.readTimeout`. Set this timeout = ~90% of 
`druid.global.http.readTimeout`|`PT4M`|
-|`druid.global.http.numMaxThreads`|Maximum number of I/O worker 
threads|`max(10, ((number of cores * 17) / 16 + 2) + 30)`|

Review Comment:
   @gianm , could you please add a short comment in the code regarding the 
heuristics behind the formula?



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