hachikuji commented on a change in pull request #8591: URL: https://github.com/apache/kafka/pull/8591#discussion_r418741006
########## File path: core/src/test/scala/unit/kafka/security/authorizer/AclEntryTest.scala ########## @@ -23,27 +23,28 @@ import org.apache.kafka.common.acl.AclOperation.READ import org.apache.kafka.common.acl.AclPermissionType.{ALLOW, DENY} import org.apache.kafka.common.security.auth.KafkaPrincipal import org.junit.{Assert, Test} -import org.scalatestplus.junit.JUnitSuite -import scala.jdk.CollectionConverters._ +import scala.collection.JavaConverters._ Review comment: Need to revert this. It is causing the build to fail. ########## File path: core/src/main/scala/kafka/utils/Json.scala ########## @@ -35,16 +35,7 @@ object Json { */ def parseFull(input: String): Option[JsonValue] = Review comment: I'm probably missing something obvious, but I found the following callers of this function: `DeleteRecordsCommand`, `LeaderElectionCommand`, `PreferredReplicaLeaderElectionCommand`, and `ReassignPartitionsCommand`. As far as I can tell, none of these uses involve the parsing of ACLs. Did we lose this compatibility handling at some point or is there some magic invocation? ---------------------------------------------------------------- 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