Re: [VOTE] KIP-228 Negative record timestamp support

2018-10-19 Thread Konstantin Chukhlomin
> > > > > > 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/K

Re: [VOTE] KIP 368: Allow SASL Connections to Periodically Re-Authenticate

2018-09-18 Thread Konstantin Chukhlomin
+1 (non binding) > On Sep 18, 2018, at 1:18 PM, michael.kamin...@nytimes.com wrote: > > > > On 2018/09/18 14:59:09, Ron Dagostino wrote: >> Hi everyone. I would like to start the vote for KIP-368: >> https://cwiki.apache.org/confluence/display/KAFKA/KIP-368%3A+Allow+SASL+Connections+to+Perio

Re: [VOTE] KIP-228 Negative record timestamp support

2018-05-29 Thread Konstantin Chukhlomin
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 > wrote: > >> All, >> &

[VOTE] KIP-228 Negative record timestamp support

2018-05-23 Thread Konstantin Chukhlomin
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

Re: [DISCUSS] KIP-228 Negative record timestamp support

2018-01-16 Thread Konstantin Chukhlomin
tore timestamp values in the payload rather than in the >>>>> metadata. >>>>> >>>>> Given that we have the current system in place, I suspect that any >>>>> translation to using Long.MIN_VALUE as the sentinel is probably just >> more >>>>> confusin

Re: [DISCUSS] KIP-228 Negative record timestamp support

2017-12-11 Thread Konstantin Chukhlomin
ive+record+timestamp+support#KIP-228Negativerecordtimestampsupport-Changesfromconsumerperspective> Let me know if it makes sense to you. -Konstantin > On Dec 7, 2017, at 2:46 PM, Konstantin Chukhlomin wrote: > > Hi Matthias, > > Indeed for consumers it will be not obvious w

Re: [DISCUSS] KIP-228 Negative record timestamp support

2017-12-07 Thread Konstantin Chukhlomin
Streams, we could check the topic config and process negative > timestamps only if the topic is configures with the new timestamp type. > > > Maybe I am a little bit to paranoid about overloading -1 semantics. > Curious to get feedback from others. > > > > -Matthias >

Re: [DISCUSS] KIP-228 Negative record timestamp support

2017-12-05 Thread Konstantin Chukhlomin
can not put timestamp in the Record Header or the message payload. Does > this sound reasonable? > > Thanks, > Dong > > > > On Tue, Dec 5, 2017 at 8:40 AM, Konstantin Chukhlomin > wrote: > >> Hi all, >> >> I have created a KIP to support negative timest

Re: [DISCUSS] KIP-228 Negative record timestamp support

2017-12-05 Thread Konstantin Chukhlomin
onInvalidTimestamp() doesn't have any check. > > What if the timestamp corresponds to 5000 BC ? Is that still allowed ? > > Cheers > > On Tue, Dec 5, 2017 at 10:29 AM, Konstantin Chukhlomin > wrote: > >> Hi Ted, >> >> Thank you for the response. >

Re: [DISCUSS] KIP-228 Negative record timestamp support

2017-12-05 Thread Konstantin Chukhlomin
ility section, there are two questions without answers. > Are you going to fill out later ? > > Cheers > > On Tue, Dec 5, 2017 at 8:40 AM, Konstantin Chukhlomin > wrote: > >> Hi all, >> >> I have created a KIP to support negative timestamp: >> https://cwik

[DISCUSS] KIP-228 Negative record timestamp support

2017-12-05 Thread Konstantin Chukhlomin
Hi all, I have created a KIP to support negative timestamp: https://cwiki.apache.org/confluence/display/KAFKA/KIP-228+Negative+record+timestamp+support Here are proposed changes: https://github.com/ap

Permission request to create KIP

2017-11-27 Thread Konstantin Chukhlomin
Hi, I'd like to create "KIP - Negative Timestamp Support". Could you please add a permission to create pages on https://cwiki.apache.org ? My email is chuhlo...@gmail.com Thanks, Konstantin