@lindong28 Hmm.. On second thought, there was a reason why I wanted to include 
the leader epoch in `PartitionInfo`. We use this object inside 
`common.Cluster`, which is depended upon by `clients.Metadata`. If we don't add 
leader epoch to `PartitionInfo`. Then we'll need to propagate the information 
through an additional argument (probably a map) to `Metadata.update`. It's a 
bit annoying and unnatural, but we can do it if we want to keep leader epoch 
hidden. Perhaps we need to introduce an intermediate object to remove our 
internal dependence on a publicly exposed object.

[ 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