jolshan commented on code in PR #15685:
URL: https://github.com/apache/kafka/pull/15685#discussion_r1613634445
##########
server-common/src/main/java/org/apache/kafka/server/common/TestFeatureVersion.java:
##########
@@ -21,16 +21,17 @@
public enum TestFeatureVersion implements FeatureVersion {
- TEST_0(0, MetadataVersion.IBP_3_3_IV0, Collections.emptyMap()),
+ // 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()));
- private short featureLevel;
- private MetadataVersion metadataVersionMapping;
- private Map<String, Short> dependencies;
+ private final short featureLevel;
+ private final MetadataVersion metadataVersionMapping;
+ private final Map<String, Short> dependencies;
public static final String FEATURE_NAME = "test.feature.version";
- public static final TestFeatureVersion PRODUCTION_VERSION = TEST_1;
+ public static final TestFeatureVersion LATEST_PRODUCTION = TEST_1;
Review Comment:
It does not since in Features, I don't define a production version. I set
this for testing purposes. But Features.PRODUCTION_VERSIONS does not contain
this feature.
--
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
