Hi Colin, Looks good. I just had one question. It sounds like your intent is to change kafka-configs.sh so that the --zookeeper flag is only supported for bootstrapping. I assume in the case of SCRAM that we will only make this change after the broker API is available?
Thanks, Jason On Tue, May 19, 2020 at 5:22 AM Mickael Maison <mickael.mai...@gmail.com> wrote: > +1 (binding) > Thanks Colin > > On Tue, May 19, 2020 at 10:57 AM Manikumar <manikumar.re...@gmail.com> > wrote: > > > > +1 (binding) > > > > Thanks for the KIP > > > > On Tue, May 19, 2020 at 12:29 PM David Jacot <dja...@confluent.io> > wrote: > > > > > +1 (non-binding). > > > > > > Thanks for the KIP. > > > > > > On Fri, May 15, 2020 at 12:41 AM Guozhang Wang <wangg...@gmail.com> > wrote: > > > > > > > +1. > > > > > > > > Thanks Colin! > > > > > > > > Guozhang > > > > > > > > On Tue, May 12, 2020 at 3:45 PM Colin McCabe <cmcc...@apache.org> > wrote: > > > > > > > > > Hi all, > > > > > > > > > > I'd like to start a vote on KIP-604: Remove ZooKeeper Flags from > the > > > > > Administrative Tools. > > > > > > > > > > As a reminder, this KIP is for the next major release of Kafka, > the 3.0 > > > > > release. So it won't go into the upcoming 2.6 release. It's a > pretty > > > > > small change that just removes the --zookeeper flags from some > tools > > > and > > > > > removes a deprecated tool. We haven't decided exactly when we'll > do > > > 3.0 > > > > > but I believe we will certainly want this change in that release. > > > > > > > > > > The KIP does contain one small change relevant to Kafka 2.6: adding > > > > > support for --if-exists and --if-not-exists in combination with the > > > > > --bootstrap-server flag. > > > > > > > > > > best, > > > > > Colin > > > > > > > > > > > > > > > > > -- > > > > -- Guozhang > > > > > > > >
