junrao commented on code in PR #16183: URL: https://github.com/apache/kafka/pull/16183#discussion_r1683517785
########## server-common/src/main/java/org/apache/kafka/server/common/MetadataVersion.java: ########## @@ -217,8 +217,11 @@ public enum MetadataVersion { // Add ELR related supports (KIP-966). IBP_3_9_IV1(22, "3.9", "IV1", true), + IBP_3_9_IV2(23, "3.9", "IV2", false), Review Comment: Should we add a new MV just for bootstrapping a new feature or just piggyback on an existing one (e.g., IBP_3_9_IV1)? > I planned to make the update where we don't send features as part of the registration/api versions if the version range is 0-0. Colin's PR only skipped a finalized version 0 feature in ApiResponse. Do you plan to do that for supported features for registration/api? Range 0-0 can now happen since some of the v1 features are not stable yet. -- 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