This was a discussion, not a vote (sorry for mangling the title), but thanks for all the +1 anyway.
Regarding Ismael's feedback: The current server.properties includes the following: # Switch to enable topic deletion or not, default value is false #delete.topic.enable=true We can't leave it as is, obviously - since the KIP invalidates the comment. Lets just remove those two lines? Note that all our proposed changes may break few community puppet/docker scripts that use these lines for "sed" that enables topic deletion. Gwen On Fri, May 26, 2017 at 5:41 PM Tom Crayford <tcrayf...@heroku.com> wrote: > +1 (non-binding) > > On Fri, May 26, 2017 at 3:38 PM, Damian Guy <damian....@gmail.com> wrote: > > > +1 > > Also agree with what Ismael said. > > > > On Fri, 26 May 2017 at 15:26 Ismael Juma <ism...@juma.me.uk> wrote: > > > > > Thanks for the KIP, sounds good to me. One comment: not sure we need to > > add > > > the config to server.properties. Do we expect people to change this > > > default? > > > > > > On Fri, May 26, 2017 at 3:03 PM, Gwen Shapira <g...@confluent.io> > wrote: > > > > > > > Hi Kafka developers, users and friends, > > > > > > > > I've added a KIP to improve our out-of-the-box usability a bit: > > > > KIP-162: Enable topic deletion by default: > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-162+-+ > > > > Enable+topic+deletion+by+default > > > > > > > > Pretty simple :) Discussion and feedback are welcome. > > > > > > > > Gwen > > > > > > > > > >
