@lindong28 Thanks for the review. I'll address your comments shortly. To answer 
your question, I think the change to `PartitionInfo` is not strictly necessary. 
I thought since we are exposing all the rest of the metadata and we expose 
leader epoch everywhere else, we may as well include it. Debugging is probably 
the only use case I can think of. Another option might be to expose it only in 
the AdminClient `TopicPartitionInfo`. We can also punt on it for now and add it 
later if we find a more compelling need. What do you think?

[ Full content available at: https://github.com/apache/kafka/pull/5564 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to