tinaselenge commented on PR #15175: URL: https://github.com/apache/kafka/pull/15175#issuecomment-2137330176
Hi @CalvinConfluent, when enabling KRaft support for Admin integration tests, one of the tests [failed](https://ci-builds.apache.org/job/Kafka/job/kafka-pr/job/PR-15175/9/#showFailuresLink) (testAuthorizedOperations) due to how DescribeTopic API request is handled (introduced by KIP-966). The test is expecting a null `authorizedOperations` if `includeAuthorizedOperations` flag is not set when describing a topic partition. In zookeeper mode, the metadata response does not contain the `authorizedOperations` for the topic partitions, if this flag is not set. However, the response from the new DescribeTopic API, it seems to always include `authorizedOperations`, despite the flag. Is this an intentional change? It wasn't clear to me when reading the KIP. I think we should stay consistent with zookeeper behaviour. If this wasn't intentional, I can raise a Jira issue for it. -- 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
