[GitHub] [kafka] mumrah commented on a diff in pull request #13631: KAFKA-14909: check zkMigrationReady tag before migration

2023-04-25 Thread via GitHub
mumrah commented on code in PR #13631: URL: https://github.com/apache/kafka/pull/13631#discussion_r1176825739 ## metadata/src/main/java/org/apache/kafka/controller/QuorumFeatures.java: ## @@ -128,4 +128,27 @@ VersionRange localSupportedFeature(String featureName) { boolean

[GitHub] [kafka] mumrah commented on a diff in pull request #13631: KAFKA-14909: check zkMigrationReady tag before migration

2023-04-24 Thread via GitHub
mumrah commented on code in PR #13631: URL: https://github.com/apache/kafka/pull/13631#discussion_r1175362654 ## metadata/src/main/java/org/apache/kafka/metadata/migration/KRaftMigrationDriver.java: ## @@ -134,8 +139,7 @@ private void initializeMigrationState() { } p