You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@kafka.apache.org by Patrick Stuedi <ps...@confluent.io.INVALID> on 2021/11/08 09:17:34 UTC

[VOTE] KIP-791: Add Record Metadata to State Store Context

Hi all,

Thanks for the feedback on KIP-791, I have updated the KIP and would like
to start the voting.

The KIP can be found here:
https://cwiki.apache.org/confluence/x/I5BnCw

Please vote in this thread.

Thanks!
-Patrick

Re: [VOTE] KIP-791: Add Record Metadata to State Store Context

Posted by Patrick Stuedi <ps...@confluent.io.INVALID>.

Thanks everyone for voting.

Voting passed after 3 days with the following +1s:

- Luke Chen
- Vasiliki (Vicky) Papavasileiou
- Guozhang Wang (binding)
- John Roesler (binding)
- Bruno Cadonna (binding)

I'll update the KIP status accordingly.

Best,
  Patrick

On Wed, Nov 10, 2021 at 5:03 PM Bruno Cadonna <ca...@apache.org> wrote:

> Patrick,
>
> Thank you for the KIP!
>
> +1 (binding)
>
> Best,
> Bruno
>
> On 09.11.21 16:58, John Roesler wrote:
> > +1 (binding) from me.
> >
> > Thanks, Patrick!
> >
> > On Mon, 2021-11-08 at 14:08 -0800, Guozhang Wang wrote:
> >> +1, thanks Patrick!
> >>
> >>
> >> Guozhang
> >>
> >> On Mon, Nov 8, 2021 at 5:44 AM Vasiliki Papavasileiou
> >> <vp...@confluent.io.invalid> wrote:
> >>
> >>> Hi Patrick,
> >>>
> >>> Having the recordMetadata available in the state stores is fundamental
> for
> >>> the consistency work and the proposed approach is reasonable.
> >>>
> >>> +1 (non-binding)
> >>>
> >>> Thank you,
> >>> Vicky
> >>>
> >>> On Mon, Nov 8, 2021 at 10:00 AM Luke Chen <sh...@gmail.com> wrote:
> >>>
> >>>> Hi Patrick,
> >>>> Thanks for the KIP.
> >>>> Adding RecordMetadata into StateStoreContext for offset updating makes
> >>>> sense to me.
> >>>>
> >>>> +1 (non-binding)
> >>>>
> >>>> Thank you.
> >>>> Luke
> >>>>
> >>>>
> >>>> On Mon, Nov 8, 2021 at 5:18 PM Patrick Stuedi
> >>> <pstuedi@confluent.io.invalid
> >>>>>
> >>>> wrote:
> >>>>
> >>>>> Hi all,
> >>>>>
> >>>>> Thanks for the feedback on KIP-791, I have updated the KIP and would
> >>> like
> >>>>> to start the voting.
> >>>>>
> >>>>> The KIP can be found here:
> >>>>> https://cwiki.apache.org/confluence/x/I5BnCw
> >>>>>
> >>>>> Please vote in this thread.
> >>>>>
> >>>>> Thanks!
> >>>>> -Patrick
> >>>>>
> >>>>
> >>>
> >>
> >>
> >
> >
>

Re: [VOTE] KIP-791: Add Record Metadata to State Store Context

Posted by Bruno Cadonna <ca...@apache.org>.

Patrick,

Thank you for the KIP!

+1 (binding)

Best,
Bruno

On 09.11.21 16:58, John Roesler wrote:
> +1 (binding) from me.
> 
> Thanks, Patrick!
> 
> On Mon, 2021-11-08 at 14:08 -0800, Guozhang Wang wrote:
>> +1, thanks Patrick!
>>
>>
>> Guozhang
>>
>> On Mon, Nov 8, 2021 at 5:44 AM Vasiliki Papavasileiou
>> <vp...@confluent.io.invalid> wrote:
>>
>>> Hi Patrick,
>>>
>>> Having the recordMetadata available in the state stores is fundamental for
>>> the consistency work and the proposed approach is reasonable.
>>>
>>> +1 (non-binding)
>>>
>>> Thank you,
>>> Vicky
>>>
>>> On Mon, Nov 8, 2021 at 10:00 AM Luke Chen <sh...@gmail.com> wrote:
>>>
>>>> Hi Patrick,
>>>> Thanks for the KIP.
>>>> Adding RecordMetadata into StateStoreContext for offset updating makes
>>>> sense to me.
>>>>
>>>> +1 (non-binding)
>>>>
>>>> Thank you.
>>>> Luke
>>>>
>>>>
>>>> On Mon, Nov 8, 2021 at 5:18 PM Patrick Stuedi
>>> <pstuedi@confluent.io.invalid
>>>>>
>>>> wrote:
>>>>
>>>>> Hi all,
>>>>>
>>>>> Thanks for the feedback on KIP-791, I have updated the KIP and would
>>> like
>>>>> to start the voting.
>>>>>
>>>>> The KIP can be found here:
>>>>> https://cwiki.apache.org/confluence/x/I5BnCw
>>>>>
>>>>> Please vote in this thread.
>>>>>
>>>>> Thanks!
>>>>> -Patrick
>>>>>
>>>>
>>>
>>
>>
> 
>

Re: [VOTE] KIP-791: Add Record Metadata to State Store Context

Posted by John Roesler <vv...@apache.org>.

+1 (binding) from me.

Thanks, Patrick!

On Mon, 2021-11-08 at 14:08 -0800, Guozhang Wang wrote:
> +1, thanks Patrick!
> 
> 
> Guozhang
> 
> On Mon, Nov 8, 2021 at 5:44 AM Vasiliki Papavasileiou
> <vp...@confluent.io.invalid> wrote:
> 
> > Hi Patrick,
> > 
> > Having the recordMetadata available in the state stores is fundamental for
> > the consistency work and the proposed approach is reasonable.
> > 
> > +1 (non-binding)
> > 
> > Thank you,
> > Vicky
> > 
> > On Mon, Nov 8, 2021 at 10:00 AM Luke Chen <sh...@gmail.com> wrote:
> > 
> > > Hi Patrick,
> > > Thanks for the KIP.
> > > Adding RecordMetadata into StateStoreContext for offset updating makes
> > > sense to me.
> > > 
> > > +1 (non-binding)
> > > 
> > > Thank you.
> > > Luke
> > > 
> > > 
> > > On Mon, Nov 8, 2021 at 5:18 PM Patrick Stuedi
> > <pstuedi@confluent.io.invalid
> > > > 
> > > wrote:
> > > 
> > > > Hi all,
> > > > 
> > > > Thanks for the feedback on KIP-791, I have updated the KIP and would
> > like
> > > > to start the voting.
> > > > 
> > > > The KIP can be found here:
> > > > https://cwiki.apache.org/confluence/x/I5BnCw
> > > > 
> > > > Please vote in this thread.
> > > > 
> > > > Thanks!
> > > > -Patrick
> > > > 
> > > 
> > 
> 
>

Re: [VOTE] KIP-791: Add Record Metadata to State Store Context

Posted by Guozhang Wang <wa...@gmail.com>.

+1, thanks Patrick!


Guozhang

On Mon, Nov 8, 2021 at 5:44 AM Vasiliki Papavasileiou
<vp...@confluent.io.invalid> wrote:

> Hi Patrick,
>
> Having the recordMetadata available in the state stores is fundamental for
> the consistency work and the proposed approach is reasonable.
>
> +1 (non-binding)
>
> Thank you,
> Vicky
>
> On Mon, Nov 8, 2021 at 10:00 AM Luke Chen <sh...@gmail.com> wrote:
>
> > Hi Patrick,
> > Thanks for the KIP.
> > Adding RecordMetadata into StateStoreContext for offset updating makes
> > sense to me.
> >
> > +1 (non-binding)
> >
> > Thank you.
> > Luke
> >
> >
> > On Mon, Nov 8, 2021 at 5:18 PM Patrick Stuedi
> <pstuedi@confluent.io.invalid
> > >
> > wrote:
> >
> > > Hi all,
> > >
> > > Thanks for the feedback on KIP-791, I have updated the KIP and would
> like
> > > to start the voting.
> > >
> > > The KIP can be found here:
> > > https://cwiki.apache.org/confluence/x/I5BnCw
> > >
> > > Please vote in this thread.
> > >
> > > Thanks!
> > > -Patrick
> > >
> >
>


-- 
-- Guozhang

Re: [VOTE] KIP-791: Add Record Metadata to State Store Context

Posted by Vasiliki Papavasileiou <vp...@confluent.io.INVALID>.

Hi Patrick,

Having the recordMetadata available in the state stores is fundamental for
the consistency work and the proposed approach is reasonable.

+1 (non-binding)

Thank you,
Vicky

On Mon, Nov 8, 2021 at 10:00 AM Luke Chen <sh...@gmail.com> wrote:

> Hi Patrick,
> Thanks for the KIP.
> Adding RecordMetadata into StateStoreContext for offset updating makes
> sense to me.
>
> +1 (non-binding)
>
> Thank you.
> Luke
>
>
> On Mon, Nov 8, 2021 at 5:18 PM Patrick Stuedi <pstuedi@confluent.io.invalid
> >
> wrote:
>
> > Hi all,
> >
> > Thanks for the feedback on KIP-791, I have updated the KIP and would like
> > to start the voting.
> >
> > The KIP can be found here:
> > https://cwiki.apache.org/confluence/x/I5BnCw
> >
> > Please vote in this thread.
> >
> > Thanks!
> > -Patrick
> >
>

Re: [VOTE] KIP-791: Add Record Metadata to State Store Context

Posted by Luke Chen <sh...@gmail.com>.

Hi Patrick,
Thanks for the KIP.
Adding RecordMetadata into StateStoreContext for offset updating makes
sense to me.

+1 (non-binding)

Thank you.
Luke

On Mon, Nov 8, 2021 at 5:18 PM Patrick Stuedi <ps...@confluent.io.invalid>
wrote:

> Hi all,
>
> Thanks for the feedback on KIP-791, I have updated the KIP and would like
> to start the voting.
>
> The KIP can be found here:
> https://cwiki.apache.org/confluence/x/I5BnCw
>
> Please vote in this thread.
>
> Thanks!
> -Patrick
>