divijvaidya commented on code in PR #12153:
URL: https://github.com/apache/kafka/pull/12153#discussion_r871713953
##########
core/src/main/scala/kafka/server/KafkaConfig.scala:
##########
@@ -894,7 +894,7 @@ object KafkaConfig {
"record batch size accepted by the broker is defined via
<code>message.max.bytes</code> (broker config) or " +
"<code>max.message.bytes</code> (topic config)."
val NumReplicaFetchersDoc = "Number of fetcher threads used to replicate
messages from a source broker. " +
- "Increasing this value can increase the degree of I/O parallelism in the
follower broker."
+ "Increasing this value can increase the degree of I/O parallelism in the
follower and leader broker at the cost of higher CPU utilization."
Review Comment:
Increasing the number of threads in a process doesn't just increase the CPU
utilization but it also consumes memory. I don't think it is necessary to
explain the cost of creating a thread in the documentation since the reader is
assumed to be having a technical background. Hence, I don't think this change
is needed.
--
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: jira-unsubscr...@kafka.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
