It's a bit subjective for sure, so feel free to ignore the suggestion. I just thought the use of -1 was a little odd. A more explicit alternative might be something like this: ``` if (latestEpoch.isEmpty || batch.partitionLeaderEpoch > latestEpoch.get) ``` I'll leave it up to you.
[ Full content available at: https://github.com/apache/kafka/pull/5320 ] This message was relayed via gitbox.apache.org for [email protected]
