jolshan commented on code in PR #15673: URL: https://github.com/apache/kafka/pull/15673#discussion_r1626234150
########## server-common/src/main/java/org/apache/kafka/server/common/TestFeatureVersion.java: ########## @@ -24,7 +24,7 @@ public enum TestFeatureVersion implements FeatureVersion { // TEST_1 released right before MV 3.7-IVO was released, and it has no dependencies TEST_1(1, MetadataVersion.IBP_3_7_IV0, Collections.emptyMap()), // TEST_2 released right before MV 3.8-IVO was released, and it depends on this metadata version - TEST_2(2, MetadataVersion.IBP_3_8_IV0, Collections.singletonMap(MetadataVersion.FEATURE_NAME, MetadataVersion.IBP_3_8_IV0.featureLevel())); + TEST_2(2, MetadataVersion.IBP_4_0_IVO, Collections.singletonMap(MetadataVersion.FEATURE_NAME, MetadataVersion.IBP_4_0_IVO.featureLevel())); Review Comment: Yeah, I was thinking about this. If we should always have this version be an unstable version. I think there are trade offs either way. We get the unstable version test coverage, but we have to keep updating it every time the MV becomes stable. -- 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