dajac commented on code in PR #18989:
URL: https://github.com/apache/kafka/pull/18989#discussion_r1966113060
##########
group-coordinator/src/main/java/org/apache/kafka/coordinator/group/GroupMetadataManager.java:
##########
@@ -2533,7 +2561,7 @@ private boolean maybeUpdateRegularExpressions(
executor.schedule(
key,
() -> refreshRegularExpressions(groupId, log, time,
metadataImage, regexes),
Review Comment:
Would it be possible to do the filtering in `refreshRegularExpressions`?
`refreshRegularExpressions` is executed in a different thread so we can take
advantage of it too and avoid having to run the authorized in the coordinator
threads.
--
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
