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