+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 > <vpapavasile...@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 <show...@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 > > <pstu...@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 > > > > > > > > > > >