[GitHub] [kafka] cmccabe commented on a diff in pull request #14169: KAFKA-15318: Update the Authorizer via AclPublisher

2023-08-09 Thread via GitHub
cmccabe commented on code in PR #14169: URL: https://github.com/apache/kafka/pull/14169#discussion_r1288918541 ## core/src/main/scala/kafka/server/metadata/AclPublisher.scala: ## @@ -0,0 +1,99 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one or more + *

[GitHub] [kafka] cmccabe commented on a diff in pull request #14169: KAFKA-15318: Update the Authorizer via AclPublisher

2023-08-09 Thread via GitHub
cmccabe commented on code in PR #14169: URL: https://github.com/apache/kafka/pull/14169#discussion_r1288849218 ## metadata/src/main/java/org/apache/kafka/controller/AclControlManager.java: ## @@ -256,11 +224,6 @@ public void replay( throw new

[GitHub] [kafka] cmccabe commented on a diff in pull request #14169: KAFKA-15318: Update the Authorizer via AclPublisher

2023-08-09 Thread via GitHub
cmccabe commented on code in PR #14169: URL: https://github.com/apache/kafka/pull/14169#discussion_r1288848313 ## metadata/src/main/java/org/apache/kafka/controller/QuorumController.java: ## @@ -1112,35 +1102,12 @@ private void appendRaftEvent(String name, Runnable runnable) {