hachikuji commented on a change in pull request #9284:
URL: https://github.com/apache/kafka/pull/9284#discussion_r496287274



##########
File path: docs/upgrade.html
##########
@@ -27,6 +27,14 @@ <h5><a id="upgrade_270_notable" 
href="#upgrade_270_notable">Notable changes in 2
         <code>default.api.timeout.ms</code>, and Kafka Streams' new 
<code>task.timeout.ms</code> parameters instead.
         Note that parameter <code>retry.backoff.ms</code> is not impacted by 
this change.
     </li>
+    <li>Altering non-reconfigurable configs of existent listeners causes 
<code>InvalidRequestException</code>.
+        By contrast, the previous behavior would have caused the updated 
configuration to be persisted, but it wouldn't
+        take effect until the broker was restarted. This change breaks 
behavior compatibility but the old behavior is not

Review comment:
       nit: can we take out the sentence about behavior compatibility. I think 
it is enough to say that the previous behavior was unintended.




----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to