Hey Thomas, Thanks for the KIP. I have few comments/questions about it:
1. It would be great if we could better explain why we need this in the motivation. At the moment, it only explains what will be added. 2. You plan to add the `MAX_TIMESTAMP` constant to tell the broker to return the maximum timestamp. How do you plan to handle the case where a new admin client would send this to an old broker which does not support it? Best, David On Tue, Apr 20, 2021 at 6:27 PM Thomas Scott <t...@confluent.io.invalid> wrote: > Hey all, > > Just a quick prod for reviews on this. I'm looking to open a vote on > Thursday if there are no objections. > > Thanks > > Tom > > > > > > Hey all, > > > > I'd like to open up the discussion on a KIP-734. This adds a new > > OffsetSpec to AdminClient.listOffsets so that we can easily determine the > > offset and timestamp of the message with the largest timestamp on a > > partition. > > > > Please give it a look over and let me know what you think. > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-734%3A+Improve+AdminClient.listOffsets+to+return+timestamp+and+offset+for+the+record+with+the+largest+timestamp > > > > Thanks > > > > Tom > > >
