cmccabe commented on code in PR #16421:
URL: https://github.com/apache/kafka/pull/16421#discussion_r1671814440
##########
core/src/main/scala/kafka/server/BrokerFeatures.scala:
##########
@@ -93,6 +93,9 @@ object BrokerFeatures extends Logging {
feature.latestProduction
}))
}
+ if (unstableFeatureVersionsEnabled) {
+ features.put("kraft.version", new SupportedVersionRange(1))
Review Comment:
> I'm a little confused here. This has one argument?
There's a constructor that creates a range 0-N based on just being given N.
I will change it to be the two-argument constructor since I guess that's
confusing.
> And is kraft.version part of this PR?
I want to have it here for test purposes. I will create it for real in a
follow-on PR, if this PR ever concludes.
--
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]
