"Each metadata version has a corresponding integer in the
MetadataVersion.java file."

Please correct me if I'm wrong, but are you referring to "featureLevel" in
the enum at
https://github.com/apache/kafka/blob/trunk/server-common/src/main/java/org/apache/kafka/server/common/MetadataVersion.java#L45
? Is yes, can we please update the description of the metric to make it
easier for the users to understand this? For example, we can say,
"Represents the current metadata version as an integer value. See
MetadataVersion (hyperlink) for a mapping between string and integer
formats of metadata version".

--
Divij Vaidya



On Tue, Jun 6, 2023 at 1:51 PM Ron Dagostino <rndg...@gmail.com> wrote:

> Thanks again for the KIP, Colin.  +1 (binding).
>
> Ron
>
> > On Jun 6, 2023, at 7:02 AM, Igor Soarez <soa...@apple.com.invalid>
> wrote:
> >
> > Thanks for the KIP.
> >
> > Seems straightforward, LGTM.
> > Non binding +1.
> >
> > --
> > Igor
> >
>

Reply via email to