rondagostino commented on code in PR #13116:
URL: https://github.com/apache/kafka/pull/13116#discussion_r1070011382


##########
metadata/src/main/java/org/apache/kafka/controller/ControllerRequestContext.java:
##########
@@ -42,19 +45,39 @@ public static OptionalLong requestTimeoutMsToDeadlineNs(
     private final OptionalLong deadlineNs;
     private final RequestHeaderData requestHeader;
 
+    private final Optional<Consumer<Double>> requestedPartitionCountRecorder;
+

Review Comment:
   We add this here because `class ControllerMutationQuotaManager` and `trait 
ControllerMutationQuota` are written in Scala while the KRaft controller and 
such are written in Java with no access to the Scala class/trait -- we can't 
pass in anything without wrapping it in Java somehow.  All we need to do is 
record a request for some number of partitions, and this matches that 
requirement precisely.



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

Reply via email to