jolshan commented on code in PR #21096:
URL: https://github.com/apache/kafka/pull/21096#discussion_r2632406196
##########
storage/src/main/java/org/apache/kafka/storage/internals/log/ProducerStateEntry.java:
##########
@@ -55,11 +55,11 @@ public ProducerStateEntry(long producerId, short
producerEpoch, int coordinatorE
firstBatchMetadata.ifPresent(batchMetadata::add);
Review Comment:
Yeah, I don't have a strong preference on exactly how it is done. I just
want to make sure that it is the case that the timestamp passed into this
method (originally) was always the same as the batch timestamp. Just want to
make sure we aren't changing the logic.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
