showuon commented on PR #12110:
URL: https://github.com/apache/kafka/pull/12110#issuecomment-1129520244

   Also, the 2 failed tests are introduced by this PR. 
   ```
   Build / JDK 17 and Scala 2.13 / 
kafka.api.PlaintextAdminIntegrationTest.testDescribeTopicsWithIds(String).quorum=kraft
   Build / JDK 17 and Scala 2.13 / 
kafka.api.PlaintextAdminIntegrationTest.testInvalidIncrementalAlterConfigs(String).quorum=kraft
   ```
   
   I checked the topicID issue, it looks like there's a race condition for 
metadata cache, so that the metadata cache will return ZERO_UUID if empty:
   
https://github.com/apache/kafka/blob/trunk/core/src/main/scala/kafka/server/metadata/KRaftMetadataCache.scala#L196
   
   Thanks.


-- 
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

Reply via email to