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

2021-11-11 Thread Patrick Stuedi
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

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

2021-11-10 Thread Bruno Cadonna
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 wrote: Hi

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

2021-11-09 Thread John Roesler
+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 > wrote: > > > Hi Patrick, > > > > Having the recordMetadata available in the state stores is fundame

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

2021-11-08 Thread Guozhang Wang
+1, thanks Patrick! Guozhang On Mon, Nov 8, 2021 at 5:44 AM Vasiliki Papavasileiou 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 > >

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

2021-11-08 Thread Vasiliki Papavasileiou
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 wrote: > Hi Patrick, > Thanks for the KIP. > Adding RecordMetadata i

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

2021-11-08 Thread Luke Chen
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 wrote: > Hi all, > > Thanks for the feedback on KIP-791, I have updated the KIP and would like > t

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

2021-11-08 Thread Patrick Stuedi
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