The current patch uses `Integer leaderEpoch = null` to indicate missing leader epoch. An alternative way is to use `int leaderEpoch = RecordBatch.NO_PARTITION_LEADER_EPOCH`. Since `RecordBatch.NO_PARTITION_LEADER_EPOCH` is used in many other places, I am wondering if it is a bit more consistent to also use `int leaderEpoch = RecordBatch.NO_PARTITION_LEADER_EPOCH` here and in `OffsetAndTimestamp`.
[ Full content available at: https://github.com/apache/kafka/pull/5564 ] This message was relayed via gitbox.apache.org for [email protected]
