Hi Mickael, Thanks for the KIP. It's a good improvement. Although it's a little weird that we introduce a "new deprecated" config, it seems like this is the best solution we can have now.
+1 (binding) from me. Thank you. Luke On Thu, May 26, 2022 at 12:21 AM Ismael Juma <ism...@juma.me.uk> wrote: > Thanks, +1 (binding). > > Ismael > > On Thu, May 19, 2022 at 5:04 AM Mickael Maison <mickael.mai...@gmail.com> > wrote: > > > Hi Ismael, > > > > That's a good idea, I've updated the KIP. > > > > Thanks, > > Mickael > > > > On Tue, May 17, 2022 at 4:17 PM Federico Valeri <fedeval...@gmail.com> > > wrote: > > > > > > +1 (non binding) > > > > > > Thanks. > > > > > > On Tue, May 17, 2022 at 3:47 PM Mickael Maison < > mickael.mai...@gmail.com> > > wrote: > > > > > > > > Hi, > > > > > > > > I'd like to start a vote on KIP-830 which proposes a method for a > > > > method for disabling JMXReporter and making JMXReporter behave like > > > > other reporters in the next major release when it will need to be > > > > explicitly listed in metric.reporters to be enabled. > > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-830%3A+Allow+disabling+JMX+Reporter > > > > > > > > Let me know if you have feedback, > > > > > > > > Thanks, > > > > Mickael > > >
