On Mon, Sep 24, 2018, at 17:49, Ismael Juma wrote:
> Thanks Colin. I think this is much needed and I'm +1 (binding)
> on fixing> this issue. However, I have a few minor suggestions:
>
> 1. Overload alterConfigs instead of creating a new method name. This
>    gives> us both the short name and a path for removal of the deprecated
> overload.
Goid idea.  +1

> 2. Did we consider Add/Remove instead of Append/Subtract?

Hmm.  I guess I was worried that Add might be confused with set.  Append
seems to better suggest adding to a multi-part entry, to me, at least.
Best,
Colin

>
> Ismael
>
> On Mon, Sep 24, 2018 at 10:29 AM Colin McCabe
> <cmcc...@apache.org> wrote:>
> > Hi all,
> >
> > I would like to start voting on KIP-339, which creates a new
> > IncrementalAlterConfigs API.
> >
> > The KIP is described here:
> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-339%3A+Create+a+new+ModifyConfigs+API>
> >  >
> > Previous discussion:
> > https://sematext.com/opensee/m/Kafka/uyzND1OYRKh2RrGba1
> >
> > best,
> > Colin
> >

Reply via email to