bump On Tue, Jun 12, 2018 at 1:48 PM Bill Bejeck <bbej...@gmail.com> wrote:
> +1 > > Thanks, > Bill > > On Sun, Jun 10, 2018 at 5:32 PM Ted Yu <yuzhih...@gmail.com> wrote: > > > +1 > > > > On Sun, Jun 10, 2018 at 2:17 PM, Matthias J. Sax <matth...@confluent.io> > > wrote: > > > > > +1 (binding) > > > > > > Thanks for the KIP. > > > > > > > > > -Matthias > > > > > > On 5/29/18 9:14 AM, Konstantin Chukhlomin wrote: > > > > Thanks, updated the KIP. > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > > 228+Negative+record+timestamp+support <https://cwiki.apache.org/ > > > confluence/display/KAFKA/KIP-228+Negative+record+timestamp+support> > > > > > > > >> On May 24, 2018, at 1:39 PM, Guozhang Wang <wangg...@gmail.com> > > wrote: > > > >> > > > >> Thanks Konstantin, I have a minor comment on the wiki page's > "Proposed > > > >> Solution" section: > > > >> > > > >> "The solution is to ignore that problem and it is a choice of the > user > > > >> to..": > > > >> > > > >> ---------- > > > >> > > > >> I'd suggest we rephrase it and admit "-1 is a special value in > Kafka, > > > which > > > >> means we do not have a valid way to express Wednesday, December 31, > > 1969 > > > >> 11:59:59 PM UTC." rather than letting users to choose how to > interpret > > > it, > > > >> since as mentioned, both at Kafka broker side, as well as at Kafka > > > client > > > >> side (Streams, Producer) we have already using this protocol to set > -1 > > > as a > > > >> special value of `unknown`, so users choosing how to interpret it > > freely > > > >> would lead to confusing behaviors. Instead, we should well document > > this > > > >> caveat, and educate users in the document that, given this > situation, > > if > > > >> you do need to express Wednesday, December 31, 1969 11:59:59 PM UTC, > > > >> consider shifting it by one millisecond. > > > >> > > > >> > > > >> Otherwise, I'm +1 on this KIP. > > > >> > > > >> > > > >> Guozhang > > > >> > > > >> > > > >> > > > >> On Wed, May 23, 2018 at 2:40 PM, Konstantin Chukhlomin < > > > chuhlo...@gmail.com> > > > >> wrote: > > > >> > > > >>> All, > > > >>> > > > >>> Thanks for the feedback on KIP-228. I've updated the KIP, and would > > > like > > > >>> to start to start a vote. > > > >>> > > > >>> KIP: https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > > >>> 228+Negative+record+timestamp+support <https://cwiki.apache.org/ > > > >>> confluence/display/KAFKA/KIP-228+Negative+record+timestamp+support> > > > >>> > > > >>> Pull Request: https://github.com/apache/kafka/pull/5072/files < > > > >>> https://github.com/apache/kafka/pull/5072/files> > > > >>> > > > >>> Discussion Thread: http://mail-archives.apache. > > > >>> org/mod_mbox/kafka-dev/201712.mbox/%3c178138FA-CA30-44A8- > > > >>> b92a-b966b8ca1...@gmail.com%3e <http://mail-archives.apache. > > > >>> org/mod_mbox/kafka-dev/201712.mbox/%3C178138FA-CA30-44A8- > > > >>> b92a-b966b8ca1...@gmail.com%3E> > > > >>> > > > >>> Thanks! > > > >> > > > >> > > > >> > > > >> > > > >> -- > > > >> -- Guozhang > > > > > > > > > > > > > > > > > -- Konstantin Chukhlomin
