Re: [VOTE] KIP-811: Add config min.repartition.purge.interval.ms to Kafka Streams

[Nick Telford]

Hi everyone,

With 3 binding +1 votes and no -1 votes, this vote passes. KIP-811 has been
Accepted.

Regards,

Nick

On Sun, 16 Jan 2022 at 15:50, Sagar  wrote:

> Hey Nick,
>
> Thanks for the KIP. I am maybe late to the game here, but maybe it might be
> beneficial to mention a couple of example scenarios on how the values of
> commit.interval.ms and this new config could affect the overall behavior .
> Like what happens if the 2 values are very similar v/s if one of them is
> very high as compared to the other. Let me know what you think.
>
> Other than that, Im +1 (non-binding).
>
> Thanks!
> Sagar.
>
> On Sun, Jan 16, 2022 at 6:17 PM Luke Chen  wrote:
>
> > Hi Nick,
> >
> > Thanks for the KIP!
> > +1 (non-binding)
> >
> > Luke
> >
> > On Sat, Jan 15, 2022 at 4:55 AM Matthias J. Sax 
> wrote:
> >
> > > +1 (binding)
> > >
> > > On 1/14/22 06:32, John Roesler wrote:
> > > > Thanks for the KIP, Nick!
> > > >
> > > > +1 (binding)
> > > >
> > > > -John
> > > >
> > > > On Fri, Jan 14, 2022, at 07:40, Bruno Cadonna wrote:
> > > >> Hi Nick,
> > > >>
> > > >> Since the title of the KIP slightly changed after the vote was
> opened
> > > >> also the link to the KIP changed as a result. This is should be a
> > > >> working link:
> > > >>
> > > >> https://cwiki.apache.org/confluence/x/JY-kCw
> > > >>
> > > >> Anyways, Thanks for the KIP!
> > > >>
> > > >> I am +1 (binding)
> > > >>
> > > >> Best,
> > > >> Bruno
> > > >>
> > > >>
> > > >>
> > > >> On 12.01.22 16:34, Nick Telford wrote:
> > > >>> Hi everyone,
> > > >>>
> > > >>> I'd like to call a vote to adopt KIP-811: Add config
> > > >>> min.repartition.purge.interval.ms to Kafka Streams
> > > >>> <
> > >
> >
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-811%3A+Add+config+min.repartition.purge.interval.ms+to+Kafka+Streams
> > > >
> > > >>> .
> > > >>>
> > > >>> Regards
> > > >>>
> > > >>> Nick Telford
> > > >>>
> > >
> >
>

Re: [VOTE] KIP-811: Add config min.repartition.purge.interval.ms to Kafka Streams

[Sagar]

Hey Nick,

Thanks for the KIP. I am maybe late to the game here, but maybe it might be
beneficial to mention a couple of example scenarios on how the values of
commit.interval.ms and this new config could affect the overall behavior .
Like what happens if the 2 values are very similar v/s if one of them is
very high as compared to the other. Let me know what you think.

Other than that, Im +1 (non-binding).

Thanks!
Sagar.

On Sun, Jan 16, 2022 at 6:17 PM Luke Chen  wrote:

> Hi Nick,
>
> Thanks for the KIP!
> +1 (non-binding)
>
> Luke
>
> On Sat, Jan 15, 2022 at 4:55 AM Matthias J. Sax  wrote:
>
> > +1 (binding)
> >
> > On 1/14/22 06:32, John Roesler wrote:
> > > Thanks for the KIP, Nick!
> > >
> > > +1 (binding)
> > >
> > > -John
> > >
> > > On Fri, Jan 14, 2022, at 07:40, Bruno Cadonna wrote:
> > >> Hi Nick,
> > >>
> > >> Since the title of the KIP slightly changed after the vote was opened
> > >> also the link to the KIP changed as a result. This is should be a
> > >> working link:
> > >>
> > >> https://cwiki.apache.org/confluence/x/JY-kCw
> > >>
> > >> Anyways, Thanks for the KIP!
> > >>
> > >> I am +1 (binding)
> > >>
> > >> Best,
> > >> Bruno
> > >>
> > >>
> > >>
> > >> On 12.01.22 16:34, Nick Telford wrote:
> > >>> Hi everyone,
> > >>>
> > >>> I'd like to call a vote to adopt KIP-811: Add config
> > >>> min.repartition.purge.interval.ms to Kafka Streams
> > >>> <
> >
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-811%3A+Add+config+min.repartition.purge.interval.ms+to+Kafka+Streams
> > >
> > >>> .
> > >>>
> > >>> Regards
> > >>>
> > >>> Nick Telford
> > >>>
> >
>

Re: [VOTE] KIP-811: Add config min.repartition.purge.interval.ms to Kafka Streams

[Luke Chen]

Hi Nick,

Thanks for the KIP!
+1 (non-binding)

Luke

On Sat, Jan 15, 2022 at 4:55 AM Matthias J. Sax  wrote:

> +1 (binding)
>
> On 1/14/22 06:32, John Roesler wrote:
> > Thanks for the KIP, Nick!
> >
> > +1 (binding)
> >
> > -John
> >
> > On Fri, Jan 14, 2022, at 07:40, Bruno Cadonna wrote:
> >> Hi Nick,
> >>
> >> Since the title of the KIP slightly changed after the vote was opened
> >> also the link to the KIP changed as a result. This is should be a
> >> working link:
> >>
> >> https://cwiki.apache.org/confluence/x/JY-kCw
> >>
> >> Anyways, Thanks for the KIP!
> >>
> >> I am +1 (binding)
> >>
> >> Best,
> >> Bruno
> >>
> >>
> >>
> >> On 12.01.22 16:34, Nick Telford wrote:
> >>> Hi everyone,
> >>>
> >>> I'd like to call a vote to adopt KIP-811: Add config
> >>> min.repartition.purge.interval.ms to Kafka Streams
> >>> <
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-811%3A+Add+config+min.repartition.purge.interval.ms+to+Kafka+Streams
> >
> >>> .
> >>>
> >>> Regards
> >>>
> >>> Nick Telford
> >>>
>

Re: [VOTE] KIP-811: Add config min.repartition.purge.interval.ms to Kafka Streams

[Matthias J. Sax]

+1 (binding)

On 1/14/22 06:32, John Roesler wrote:

Thanks for the KIP, Nick!

+1 (binding)

-John

On Fri, Jan 14, 2022, at 07:40, Bruno Cadonna wrote:

Hi Nick,

Since the title of the KIP slightly changed after the vote was opened
also the link to the KIP changed as a result. This is should be a
working link:

https://cwiki.apache.org/confluence/x/JY-kCw

Anyways, Thanks for the KIP!

I am +1 (binding)

Best,
Bruno



On 12.01.22 16:34, Nick Telford wrote:

Hi everyone,

I'd like to call a vote to adopt KIP-811: Add config
min.repartition.purge.interval.ms to Kafka Streams

.

Regards

Nick Telford

Re: [VOTE] KIP-811: Add config min.repartition.purge.interval.ms to Kafka Streams

[John Roesler]

Thanks for the KIP, Nick!

+1 (binding)

-John

On Fri, Jan 14, 2022, at 07:40, Bruno Cadonna wrote:
> Hi Nick,
>
> Since the title of the KIP slightly changed after the vote was opened 
> also the link to the KIP changed as a result. This is should be a 
> working link:
>
> https://cwiki.apache.org/confluence/x/JY-kCw
>
> Anyways, Thanks for the KIP!
>
> I am +1 (binding)
>
> Best,
> Bruno
>
>
>
> On 12.01.22 16:34, Nick Telford wrote:
>> Hi everyone,
>> 
>> I'd like to call a vote to adopt KIP-811: Add config
>> min.repartition.purge.interval.ms to Kafka Streams
>> 
>> .
>> 
>> Regards
>> 
>> Nick Telford
>>

Re: [VOTE] KIP-811: Add config min.repartition.purge.interval.ms to Kafka Streams

[Bruno Cadonna]

Hi Nick,

Since the title of the KIP slightly changed after the vote was opened 
also the link to the KIP changed as a result. This is should be a 
working link:


https://cwiki.apache.org/confluence/x/JY-kCw

Anyways, Thanks for the KIP!

I am +1 (binding)

Best,
Bruno



On 12.01.22 16:34, Nick Telford wrote:

Hi everyone,

I'd like to call a vote to adopt KIP-811: Add config
min.repartition.purge.interval.ms to Kafka Streams

.

Regards

Nick Telford

[VOTE] KIP-811: Add config min.repartition.purge.interval.ms to Kafka Streams

[Nick Telford]

Hi everyone,

I'd like to call a vote to adopt KIP-811: Add config
min.repartition.purge.interval.ms to Kafka Streams

.

Regards

Nick Telford