+1 on Jun's suggestion to use "beginning" and "end". The term "latest" is misleading since the last message in the log may not have the largest timestamp.
On Mon, Sep 19, 2016 at 9:49 AM, Jun Rao <j...@confluent.io> wrote: > Hi, Jiangjie, > > Thanks for the proposal. Looks good to me overall. Just a couple of minor > comments. > > 1. I thought at some point you considered to only return offset in > offsetsForTimes instead of offset and timestamp. One benefit of doing that > is that it will make the return type consistent among all three new apis. > It also feels a bit weird that we only return timestamp, but not other > metadata associated with a message. > > 2. To be consistent with existing seek apis, would it be better to rename > earliestOffsets() and latestOffsets() to beginningOffsets() and > endOffsets()? > > Jun > > On Fri, Sep 9, 2016 at 4:38 PM, Becket Qin <becket....@gmail.com> wrote: > > > Hi all, > > > > I'd like to start the voting for KIP-79 > > > > In short we propose to : > > 1. add a ListOffsetRequest/ListOffsetResponse v1, and > > 2. add earliestOffsts(), latestOffsets() and offsetForTime() methods in > the > > new consumer. > > > > The KIP wiki is the following: > > https://cwiki.apache.org/confluence/pages/viewpage. > action?pageId=65868090 > > > > Thanks, > > > > Jiangjie (Becket) Qin > > >
