ahuang98 commented on code in PR #19545:
URL: https://github.com/apache/kafka/pull/19545#discussion_r2058913387


##########
metadata/src/main/java/org/apache/kafka/metadata/bootstrap/BootstrapMetadata.java:
##########
@@ -117,7 +117,7 @@ public List<ApiMessageAndVersion> records() {
     }
 
     public MetadataVersion metadataVersion() {
-        return metadataVersion;
+        return MetadataVersion.fromFeatureLevel(metadataVersionLevel);

Review Comment:
   got it - and there's no check that the bootstrap metadata file / bootstrap 
MV matches across all controllers? in this data loss scenario, it would work to 
re-format the directory with the current MV that the cluster is on before 
re-adding the voter?



-- 
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: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to