mjsax commented on code in PR #18087:
URL: https://github.com/apache/kafka/pull/18087#discussion_r1874111595


##########
docs/streams/developer-guide/config-streams.html:
##########
@@ -231,9 +231,9 @@ <h4><a class="toc-backref" 
href="#id23">num.standby.replicas</a><a class="header
         <p>Here are the optional <a 
href="/{{version}}/javadoc/org/apache/kafka/streams/StreamsConfig.html">Streams</a>
 javadocs, sorted by level of importance:</p>
         <blockquote>
           <div><ul class="simple">
-            <li>High: These parameters can have a significant impact on 
performance. Take care when deciding the values of these parameters.</li>
-            <li>Medium: These parameters can have some impact on performance. 
Your specific environment will determine how much tuning effort should be 
focused on these parameters.</li>
-            <li>Low: These parameters have a less general or less significant 
impact on performance.</li>
+            <li>High: These are parameters with default value which are most 
likely not a good fit for production use. It's highly recommended to revisit 
these parameters for production usage.</li>
+            <li>Medium: The default values of these parameters should work for 
production for many cases, but it's not uncommon that they are changed, for 
example to tune performance.</li>
+            <li>Low: It should be rarely necessary to change the default for 
these parameters. It's only recommended to change them if there is a very 
specific issue you want to address.</li>

Review Comment:
   Many configs have nothing to do with performance. Trying to make the level 
description more useful / actionable to users.



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

Reply via email to