You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kafka.apache.org by Colin McCabe <cm...@apache.org> on 2019/08/01 22:06:30 UTC
Re: [DISCUSS] KIP-499 - Unify connection name flag for command line tool
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 <ja...@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 <mi...@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
> > >
> >
>
Re: [DISCUSS] KIP-499 - Unify connection name flag for command line tool
Posted by Harsha <ka...@harsha.io>.
+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 <ja...@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 <mi...@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
> > > >
> > >
> >
>