cmccabe commented on code in PR #14115: URL: https://github.com/apache/kafka/pull/14115#discussion_r1276615552
########## metadata/src/main/java/org/apache/kafka/metadata/migration/KRaftMigrationDriver.java: ########## @@ -297,6 +297,16 @@ private boolean isValidStateChange(MigrationDriverState newState) { } } + private boolean checkDriverState(MigrationDriverState expectedState) { + if (migrationState.equals(expectedState)) { + return true; + } else { + log.debug("Expected driver state {} but found {}. Not running this event {}.", Review Comment: Should this be `info`? It won't happen often, will it? -- 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