Hi! I had a look at the KIP-932, and it seems KafkaShareConsumer is to be configured the same way as the normal consumer using key-value props. As I understand correctly, no adaptation is needed for it to benefit from KIP-899?
Meanwhile, the PR [1] is open for review. If there are comments that require changes, we can address them in the PR or in case it's already merged, afterwards. Best, Ivan [1] https://github.com/apache/kafka/pull/13277 On Thu, May 16, 2024, at 01:52, Jun Rao wrote: > 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 > > > > > >>> > > > > > > > > > > > > > > > > > > > > >
