Hey Nick, Thanks for the KIP. I am maybe late to the game here, but maybe it might be beneficial to mention a couple of example scenarios on how the values of commit.interval.ms and this new config could affect the overall behavior . Like what happens if the 2 values are very similar v/s if one of them is very high as compared to the other. Let me know what you think.
Other than that, Im +1 (non-binding). Thanks! Sagar. On Sun, Jan 16, 2022 at 6:17 PM Luke Chen <show...@gmail.com> wrote: > Hi Nick, > > Thanks for the KIP! > +1 (non-binding) > > Luke > > On Sat, Jan 15, 2022 at 4:55 AM Matthias J. Sax <mj...@apache.org> wrote: > > > +1 (binding) > > > > On 1/14/22 06:32, John Roesler wrote: > > > Thanks for the KIP, Nick! > > > > > > +1 (binding) > > > > > > -John > > > > > > On Fri, Jan 14, 2022, at 07:40, Bruno Cadonna wrote: > > >> Hi Nick, > > >> > > >> Since the title of the KIP slightly changed after the vote was opened > > >> also the link to the KIP changed as a result. This is should be a > > >> working link: > > >> > > >> https://cwiki.apache.org/confluence/x/JY-kCw > > >> > > >> Anyways, Thanks for the KIP! > > >> > > >> I am +1 (binding) > > >> > > >> Best, > > >> Bruno > > >> > > >> > > >> > > >> On 12.01.22 16:34, Nick Telford wrote: > > >>> Hi everyone, > > >>> > > >>> I'd like to call a vote to adopt KIP-811: Add config > > >>> min.repartition.purge.interval.ms to Kafka Streams > > >>> < > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-811%3A+Add+config+min.repartition.purge.interval.ms+to+Kafka+Streams > > > > > >>> . > > >>> > > >>> Regards > > >>> > > >>> Nick Telford > > >>> > > >
