junrao commented on PR #18997: URL: https://github.com/apache/kafka/pull/18997#issuecomment-2679825519
> I think it's cleanest to leave the metadata version unset until we actually read it from the log. That's safer. The only question is what to do in `FeatureControlManager.metadataVersion()` when MV is not set yet. Returning an option requires every caller to check it. Perhaps we could throw an exception since the caller is not expected to use this method when MV is not set yet. -- 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]
