junrao commented on code in PR #23665: URL: https://github.com/apache/kafka/pull/23665#discussion_r4319420510
########## clients/src/main/resources/common/message/RequestHeader.json: ########## @@ -25,6 +25,14 @@ // Version 2 is the first flexible version. // // Version 3 introduces ClientInstanceId. (KIP-1313) + // + // Compatibility constraints: + // - New header fields must be tagged until every broker version that clients are required to Review Comment: This is not very accurate. We can add non-tagged fields to the header as long as it's not used by ApiVersionRequest. -- 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]
