dajac commented on code in PR #16158:
URL: https://github.com/apache/kafka/pull/16158#discussion_r1631388315
##########
core/src/test/scala/unit/kafka/server/ConsumerGroupDescribeRequestsTest.scala:
##########
@@ -116,6 +115,7 @@ class ConsumerGroupDescribeRequestsTest(cluster:
ClusterInstance) extends GroupC
val timeoutMs = 5 * 60 * 1000
val clientId = "client-id"
val clientHost = "/127.0.0.1"
+ val authorizedOperationsInt = 328; // Integer representation of the
authorized operations for this request
Review Comment:
I wonder if we should replace `328` by something like
```
Utils.to32BitField(
AclEntry.supportedOperations(ResourceType.CLUSTER).asScala
.map(_.code.asInstanceOf[JByte]).asJava)
```
What do you think? We could also do it in the other place.
--
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
