kafka-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Harsha <ka...@harsha.io>
Subject Re: [DISCUSS] KIP-499 - Unify connection name flag for command line tool
Date Thu, 01 Aug 2019 22:14:02 GMT
+1 for the KIP. 
-Harsha

On Thu, Aug 1, 2019, at 3:07 PM, Colin McCabe wrote:
> On Wed, Jul 31, 2019, at 05:26, Mitchell wrote:
> > Hi Jason,
> > Thanks for looking at this!
> > 
> > I wasn't exactly sure what to put in the compatibility section.  I wrote
> > the KIP thinking that we should probably mark the old arguments for
> > deprecation for a release or two before actually removing them.  I'm happy
> > to change this either way if it's preferred.
> 
> I think Jason was proposing deprecating (but NOT removing) the old 
> arguments in the next release.
> 
> Thanks for tackling this.  +1 for the KIP.
> 
> best,
> Colin
> 
> > -mitch
> > 
> > On Tue, Jul 30, 2019 at 11:55 PM Jason Gustafson <jason@confluent.io> wrote:
> > 
> > > Hey Mitch, thanks for the KIP! This command line inconsistency frustrates
> > > me almost every day. I'm definitely +1 on this.
> > >
> > > One minor nitpick. The compatibility section mentions there will be no
> > > deprecations, but it sounds like we are planning on deprecating the old
> > > arguments?
> > >
> > > Thanks,
> > > Jason
> > >
> > > On Tue, Jul 30, 2019 at 5:25 PM Mitchell <mitchellh@gmail.com> wrote:
> > >
> > > > Hello,
> > > > I have written a proposal to add the command line argument
> > > > `--bootstrap-server` to 5 of the existing command line tools that do not
> > > > currently use `--broker-list` for passing cluster connection information.
> > > >
> > > >
> > > >
> > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-499+-+Unify+connection+name+flag+for+command+line+tool
> > > >
> > > > Please take a look and let me know what you think.
> > > > Thanks,
> > > > -Mitch
> > > >
> > >
> >
>

Mime
View raw message