Hi, Ivan, You are right. StreamsConfigs can take all existing consumer configs, with or without prefixes. So, we don't need to add the new config to StreamsConfig explicitly.
For KIP-932, it says for each new consumer config, we need to determine whether it should be added to ShareConsumer config too. Thanks, Jun On Wed, May 15, 2024 at 12:16 PM Ivan Yurchenko <i...@ivanyu.me> wrote: > Hi Jun, > > Thank you for you comment. I was thinking that this > `metadata.recovery.strategy` could be passed to the relevant consumer in > streams using the `restore.consumer.` prefix. I that what you meant or I > misunderstand? > As for the KIP-932, I'll have a closer look. > > Ivan > > > On Wed, May 15, 2024, at 20:14, Jun Rao wrote: > > Hi, Ivan, > > > > Thanks for the KIP. +1 > > > > Just a minor comment. Should we add metadata.recovery.strategy to the > > Streams and the newly introduced ShareConsumer (KIP-932) too? > > > > Jun > > > > On Wed, May 8, 2024 at 11:35 AM Manikumar <manikumar.re...@gmail.com> > wrote: > > > > > Thanks for the KIP. > > > > > > +1 (binding). > > > > > > On Wed, Apr 17, 2024 at 7:50 PM Omnia Ibrahim <o.g.h.ibra...@gmail.com > > > > > wrote: > > > > > > > > Hi Ivan, > > > > Thanks for the KIP this is a very nice feature to have. > > > > +1(non-binding) > > > > Omnia > > > > > On 15 Apr 2024, at 14:33, Andrew Schofield < > andrew_schofi...@live.com> > > > wrote: > > > > > > > > > > Thanks for the KIP > > > > > > > > > > +1 (non-binding) > > > > > > > > > > Andrew > > > > > > > > > >> On 15 Apr 2024, at 14:16, Chris Egerton <chr...@aiven.io.INVALID> > > > wrote: > > > > >> > > > > >> Hi Ivan, > > > > >> > > > > >> Thanks for the KIP. After the recent changes, this LGTM. +1 > (binding) > > > > >> > > > > >> Cheers, > > > > >> > > > > >> Chris > > > > >> > > > > >> On Wed, Aug 2, 2023 at 12:15 AM Ivan Yurchenko < > > > ivan0yurche...@gmail.com> > > > > >> wrote: > > > > >> > > > > >>> Hello, > > > > >>> > > > > >>> The discussion [1] for KIP-899 [2] has been open for quite some > > > time. I'd > > > > >>> like to put the KIP up for a vote. > > > > >>> > > > > >>> Best, > > > > >>> Ivan > > > > >>> > > > > >>> [1] > https://lists.apache.org/thread/m0ncbmfxs5m87sszby2jbmtjx2bdpcdl > > > > >>> [2] > > > > >>> > > > > >>> > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-899%3A+Allow+producer+and+consumer+clients+to+rebootstrap > > > > >>> > > > > > > > > > > > > > > >
