dengziming opened a new pull request #9769:
URL: https://github.com/apache/kafka/pull/9769


   *More detailed description of your change*
   1. Add topicNames in `MetadataCache` and add alter 
`KafkaApis.handleTopicMetadataRequest` according to the KIP
   2. Add method `describeTopicsWithIds` in KafkaAdminClient, similar to 
`describeTopics`
   3. Change `DescribeTopicsResult` to `DescribeTopicsResult<T>`, which is a 
little different from the KIP
   
   *Summary of testing strategy (including rationale)*
   Test locally, here is some result:
   ```
   New server + new Client 
   --describe --zookeeper localhost:2181 --topic-id a-Paxi-LSka9sIVy9UFp_Q
   Exception in thread "main" java.lang.IllegalArgumentException: --topic-id 
can used only with --bootstrap-server
   
   --describe --bootstrap-server localhost:9092 --topic-id 
a-Paxi-LSka9sIVy9UFp_Q
   Topic: old-version-topic     TopicId: a-Paxi-LSka9sIVy9UFp_Q PartitionCount: 
2       ReplicationFactor: 1    Configs: segment.bytes=1073741824
        Topic: old-version-topic        Partition: 0    Leader: 0       
Replicas: 0     Isr: 0
        Topic: old-version-topic        Partition: 1    Leader: 0       
Replicas: 0     Isr: 0
   
   Old Server/ new Client
   --describe --bootstrap-server localhost:9092 --topic-id 
a-Paxi-LSka9sIVy9UFp_Q
   Error while executing topic command : TopicId a-Paxi-LSka9sIVy9UFp_Q not 
found.
   
   New server + old client
   --describe --bootstrap-server localhost:9092 --topic-id 
a-Paxi-LSka9sIVy9UFp_Q
   Exception in thread "main" joptsimple.UnrecognizedOptionException: topic-id 
is not a recognized option
   ```
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


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


Reply via email to