Bringing up this minor KIP. The batch-size option is unused in the console producer. We should either remove or deprecate it.
PR: https://github.com/apache/kafka/pull/10202/ On Fri, Apr 30, 2021 at 5:32 PM Kamal Chandraprakash < kamal.chandraprak...@gmail.com> wrote: > Bumping up this thread. > > On Wed, Mar 17, 2021 at 7:44 PM Manikumar <manikumar.re...@gmail.com> > wrote: > >> Hi Kamal, >> >> It looks like we just forgot this config, when we removed old producer >> code. I think we dont require KIP for this. >> we can directly fix with a minor PR . >> >> Thanks. >> >> On Wed, Mar 17, 2021 at 7:02 PM Dongjin Lee <dong...@apache.org> wrote: >> >> > +1. (non-binding) >> > >> > Thanks, >> > Dongjin >> > >> > On Thu, Mar 11, 2021 at 5:52 PM Manikumar <manikumar.re...@gmail.com> >> > wrote: >> > >> > > +1 (binding). Thanks for the KIP >> > > I think we can remove the config option as the config option is >> unused. >> > > >> > > On Wed, Mar 10, 2021 at 3:06 PM Kamal Chandraprakash < >> > > kamal.chandraprak...@gmail.com> wrote: >> > > >> > > > Hi, >> > > > >> > > > I'd like to start a vote on KIP-717 to remove batch-size config from >> > the >> > > > console producer. >> > > > >> > > > https://cwiki.apache.org/confluence/x/DB1RCg >> > > > >> > > > Thanks, >> > > > Kamal >> > > > >> > > >> > -- >> > *Dongjin Lee* >> > >> > *A hitchhiker in the mathematical world.* >> > >> > >> > >> > *github: <http://goog_969573159/>github.com/dongjinleekr >> > <https://github.com/dongjinleekr>keybase: >> https://keybase.io/dongjinleekr >> > <https://keybase.io/dongjinleekr>linkedin: >> kr.linkedin.com/in/dongjinleekr >> > <https://kr.linkedin.com/in/dongjinleekr>speakerdeck: >> > speakerdeck.com/dongjin >> > <https://speakerdeck.com/dongjin>* >> > >> >
