chia7712 commented on code in PR #18016:
URL: https://github.com/apache/kafka/pull/18016#discussion_r1880941803


##########
metadata/src/main/java/org/apache/kafka/controller/QuorumController.java:
##########
@@ -1250,8 +1248,7 @@ private void replay(ApiMessage message, 
Optional<OffsetAndEpoch> snapshotId, lon
                 // NoOpRecord is an empty record and doesn't need to be 
replayed
                 break;
             case ZK_MIGRATION_STATE_RECORD:
-                featureControl.replay((ZkMigrationStateRecord) message);
-                break;
+                throw new UnsupportedOperationException("ZK migration is no 
longer supported.");

Review Comment:
   > For that I prefer a comment
   
   That's okay with me. We can leave comments in the ZkMigrationRecord.json 
file to remind developers about its usage, even though it has no references in 
the production code.



-- 
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]

Reply via email to