dengziming commented on a change in pull request #10289:
URL: https://github.com/apache/kafka/pull/10289#discussion_r594127628
##########
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:
It's a bit difficult to figure out how to add the window, we could not
simply rely on a fixed configuration, I add a ticket to track this problem:
https://issues.apache.org/jira/browse/KAFKA-12465.
----------------------------------------------------------------
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
