Hi Justine, On Tue, Apr 9, 2024 at 4:19 PM Justine Olshan <jols...@confluent.io.invalid> wrote: > As for the validation criteria. It seems like one bit of code that > validates whether a version is allowed is in the method > `reasonNotSupported` which checks the range of features available for the > given feature. > For metadata.version we have a method to do "additional checks" and we > could have those for the various other features as well. I have an > (internal) FeatureVersion interface in mind that would work well for this. > For any of these validations, we return the same error > `INVALID_UPDATE_VERSION`. I would think continuing to use this error > follows naturally, but if we think it is necessary to specify the error > code, I can do so in my KIP.
Thanks for looking into this. The updates to the KIP look good to me. -- -José