junrao commented on code in PR #17886:
URL: https://github.com/apache/kafka/pull/17886#discussion_r1857408051
##########
server-common/src/main/java/org/apache/kafka/server/common/Features.java:
##########
@@ -41,10 +57,20 @@ public enum Features {
*
* See {@link TestFeatureVersion} as an example. See {@link
FeatureVersion} when implementing a new feature.
*/
- TEST_VERSION("test.feature.version", TestFeatureVersion.values()),
- KRAFT_VERSION("kraft.version", KRaftVersion.values()),
- TRANSACTION_VERSION("transaction.version", TransactionVersion.values()),
- GROUP_VERSION("group.version", GroupVersion.values());
+ TEST_VERSION("test.feature.version", TestFeatureVersion.values(),
TestFeatureVersion.LATEST_PRODUCTION),
Review Comment:
Well, `Features.FEATURES` contains many features, but the enum `Features`
represents a single feature. For example, `Features.name()` provides the name
of a single 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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
