jsancio commented on a change in pull request #10289: URL: https://github.com/apache/kafka/pull/10289#discussion_r594490953
########## File path: raft/src/main/java/org/apache/kafka/raft/KafkaRaftClient.java ########## @@ -939,12 +951,27 @@ private FetchResponseData buildEmptyFetchResponse( ); } - private boolean hasValidClusterId(FetchRequestData request) { + private boolean hasValidClusterId(ApiMessage request) { Review comment: We can implement that when handling a response, invalid cluster id are fatal unless a previous response contained a valid cluster id. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org