MarcoLotz commented on a change in pull request #10131:
URL: https://github.com/apache/kafka/pull/10131#discussion_r608024597



##########
File path: docs/upgrade.html
##########
@@ -37,6 +37,8 @@ <h5><a id="upgrade_300_notable" 
href="#upgrade_300_notable">Notable changes in 3
     <li>The deprecated <code>kafka-preferred-replica-election</code> command 
line tool was removed. Please use <code>kafka-leader-election</code> 
instead.</li>
     <li>The deprecated <code>ConfigEntry</code> constructor was removed (<a 
href="https://issues.apache.org/jira/browse/KAFKA-12577";>KAFKA-12577</a>).
         Please use the remaining public constructor instead.</li>
+    <li>Kafka Streams no longer has a compile time dependency on 
"connect:json" module (<a 
href="https://issues.apache.org/jira/browse/KAFKA-5146";>KAFKA-5146</a>).

Review comment:
       In docs/upgrade.html they all reference a Kafka jira ticket with the 
hyperlink. In docs/streams/upgrade-guide.html they all reference a KIP, but 
there were previous upgrades referencing a jira ticket with the hyperlink. 
Seems consistent to keep hyperlink in both files.




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