Re: [VOTE] KIP-162: Enable topic deletion by default

[Gwen Shapira]

ah, thanks guys! (and thanks Ismael for noticing that a bunch of people
voted on the discuss thread by mistake).

We have 6 binding votes and 5 non-binding. So this update is in :)

I'll update the wiki accordingly.

On Wed, Jul 19, 2017 at 5:11 PM Becket Qin <becket@gmail.com> wrote:

> +1
>
> On Wed, Jul 19, 2017 at 11:40 AM, Jason Gustafson <ja...@confluent.io>
> wrote:
>
> > +1
> >
> > On Wed, Jul 19, 2017 at 7:51 AM, Damian Guy <damian@gmail.com>
> wrote:
> >
> > > +1
> > >
> > > On Tue, 6 Jun 2017 at 18:19 BigData dev <bigdatadev...@gmail.com>
> wrote:
> > >
> > > > +1 (non-binding)
> > > >
> > > > Thanks,
> > > > Bharat
> > > >
> > > > On Tue, Jun 6, 2017 at 9:21 AM, Ashwin Sinha <
> ashwin.si...@go-jek.com>
> > > > wrote:
> > > >
> > > > > +1
> > > > >
> > > > > On Tue, Jun 6, 2017 at 11:20 PM, Mickael Maison <
> > > > mickael.mai...@gmail.com>
> > > > > wrote:
> > > > >
> > > > > > +1 (non binding), thanks
> > > > > >
> > > > > > On Tue, Jun 6, 2017 at 2:16 PM, Bill Bejeck <bbej...@gmail.com>
> > > wrote:
> > > > > > > +1
> > > > > > >
> > > > > > > -Bill
> > > > > > >
> > > > > > > On Tue, Jun 6, 2017 at 9:08 AM, Ismael Juma <ism...@juma.me.uk
> >
> > > > wrote:
> > > > > > >
> > > > > > >> Thanks for the KIP, Gwen. +1 (binding).
> > > > > > >>
> > > > > > >> Ismael
> > > > > > >>
> > > > > > >> On Tue, Jun 6, 2017 at 5:37 AM, Gwen Shapira <
> g...@confluent.io
> > >
> > > > > wrote:
> > > > > > >>
> > > > > > >> > Hi,
> > > > > > >> >
> > > > > > >> > The discussion has been quite positive, so I posted a JIRA,
> a
> > PR
> > > > and
> > > > > > >> > updated the KIP with the latest decisions.
> > > > > > >> >
> > > > > > >> > Lets officially vote on the KIP:
> > > > > > >> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > > > > >> > 162+-+Enable+topic+deletion+by+default
> > > > > > >> >
> > > > > > >> > JIRA is here: https://issues.apache.org/
> > jira/browse/KAFKA-5384
> > > > > > >> >
> > > > > > >> > Gwen
> > > > > > >> >
> > > > > > >>
> > > > > >
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Thanks and Regards,
> > > > > Ashwin
> > > > >
> > > >
> > >
> >
>

Re: [VOTE] KIP-162: Enable topic deletion by default

[Becket Qin]

+1

On Wed, Jul 19, 2017 at 11:40 AM, Jason Gustafson <ja...@confluent.io>
wrote:

> +1
>
> On Wed, Jul 19, 2017 at 7:51 AM, Damian Guy <damian@gmail.com> wrote:
>
> > +1
> >
> > On Tue, 6 Jun 2017 at 18:19 BigData dev <bigdatadev...@gmail.com> wrote:
> >
> > > +1 (non-binding)
> > >
> > > Thanks,
> > > Bharat
> > >
> > > On Tue, Jun 6, 2017 at 9:21 AM, Ashwin Sinha <ashwin.si...@go-jek.com>
> > > wrote:
> > >
> > > > +1
> > > >
> > > > On Tue, Jun 6, 2017 at 11:20 PM, Mickael Maison <
> > > mickael.mai...@gmail.com>
> > > > wrote:
> > > >
> > > > > +1 (non binding), thanks
> > > > >
> > > > > On Tue, Jun 6, 2017 at 2:16 PM, Bill Bejeck <bbej...@gmail.com>
> > wrote:
> > > > > > +1
> > > > > >
> > > > > > -Bill
> > > > > >
> > > > > > On Tue, Jun 6, 2017 at 9:08 AM, Ismael Juma <ism...@juma.me.uk>
> > > wrote:
> > > > > >
> > > > > >> Thanks for the KIP, Gwen. +1 (binding).
> > > > > >>
> > > > > >> Ismael
> > > > > >>
> > > > > >> On Tue, Jun 6, 2017 at 5:37 AM, Gwen Shapira <g...@confluent.io
> >
> > > > wrote:
> > > > > >>
> > > > > >> > Hi,
> > > > > >> >
> > > > > >> > The discussion has been quite positive, so I posted a JIRA, a
> PR
> > > and
> > > > > >> > updated the KIP with the latest decisions.
> > > > > >> >
> > > > > >> > Lets officially vote on the KIP:
> > > > > >> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > > > >> > 162+-+Enable+topic+deletion+by+default
> > > > > >> >
> > > > > >> > JIRA is here: https://issues.apache.org/
> jira/browse/KAFKA-5384
> > > > > >> >
> > > > > >> > Gwen
> > > > > >> >
> > > > > >>
> > > > >
> > > >
> > > >
> > > >
> > > > --
> > > > Thanks and Regards,
> > > > Ashwin
> > > >
> > >
> >
>

Re: [VOTE] KIP-162: Enable topic deletion by default

[Jason Gustafson]

+1

On Wed, Jul 19, 2017 at 7:51 AM, Damian Guy <damian@gmail.com> wrote:

> +1
>
> On Tue, 6 Jun 2017 at 18:19 BigData dev <bigdatadev...@gmail.com> wrote:
>
> > +1 (non-binding)
> >
> > Thanks,
> > Bharat
> >
> > On Tue, Jun 6, 2017 at 9:21 AM, Ashwin Sinha <ashwin.si...@go-jek.com>
> > wrote:
> >
> > > +1
> > >
> > > On Tue, Jun 6, 2017 at 11:20 PM, Mickael Maison <
> > mickael.mai...@gmail.com>
> > > wrote:
> > >
> > > > +1 (non binding), thanks
> > > >
> > > > On Tue, Jun 6, 2017 at 2:16 PM, Bill Bejeck <bbej...@gmail.com>
> wrote:
> > > > > +1
> > > > >
> > > > > -Bill
> > > > >
> > > > > On Tue, Jun 6, 2017 at 9:08 AM, Ismael Juma <ism...@juma.me.uk>
> > wrote:
> > > > >
> > > > >> Thanks for the KIP, Gwen. +1 (binding).
> > > > >>
> > > > >> Ismael
> > > > >>
> > > > >> On Tue, Jun 6, 2017 at 5:37 AM, Gwen Shapira <g...@confluent.io>
> > > wrote:
> > > > >>
> > > > >> > Hi,
> > > > >> >
> > > > >> > The discussion has been quite positive, so I posted a JIRA, a PR
> > and
> > > > >> > updated the KIP with the latest decisions.
> > > > >> >
> > > > >> > Lets officially vote on the KIP:
> > > > >> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > > >> > 162+-+Enable+topic+deletion+by+default
> > > > >> >
> > > > >> > JIRA is here: https://issues.apache.org/jira/browse/KAFKA-5384
> > > > >> >
> > > > >> > Gwen
> > > > >> >
> > > > >>
> > > >
> > >
> > >
> > >
> > > --
> > > Thanks and Regards,
> > > Ashwin
> > >
> >
>

Re: KIP-162: Enable topic deletion by default

[Bill Bejeck]

+1

On Wed, Jul 19, 2017 at 1:35 PM, Jay Kreps <j...@confluent.io> wrote:

> +1
>
> On Sat, May 27, 2017 at 11:04 AM, Gwen Shapira <g...@confluent.io> wrote:
>
> > Thanks Vahid,
> >
> > Do you mind if we leave the command-line out of scope for this?
> >
> > I can see why adding confirmations, options to bypass confirmations, etc
> > would be an improvement. However, I've seen no complaints about the
> current
> > behavior of the command-line and the KIP doesn't change it at all. So I'd
> > rather address things separately.
> >
> > Gwen
> >
> > On Fri, May 26, 2017 at 8:10 PM Vahid S Hashemian <
> > vahidhashem...@us.ibm.com>
> > wrote:
> >
> > > Gwen, thanks for the KIP.
> > > It looks good to me.
> > >
> > > Just a minor suggestion: It would be great if the command asks for a
> > > confirmation (y/n) before deleting the topic (similar to how removing
> > ACLs
> > > works).
> > >
> > > Thanks.
> > > --Vahid
> > >
> > >
> > >
> > > From:   Gwen Shapira <g...@confluent.io>
> > > To: "dev@kafka.apache.org" <dev@kafka.apache.org>, Users
> > > <us...@kafka.apache.org>
> > > Date:   05/26/2017 07:04 AM
> > > Subject:        KIP-162: Enable topic deletion by default
> > >
> > >
> > >
> > > 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
> > >
> > >
> > >
> > >
> > >
> >
>

Re: KIP-162: Enable topic deletion by default

[Jay Kreps]

+1

On Sat, May 27, 2017 at 11:04 AM, Gwen Shapira <g...@confluent.io> wrote:

> Thanks Vahid,
>
> Do you mind if we leave the command-line out of scope for this?
>
> I can see why adding confirmations, options to bypass confirmations, etc
> would be an improvement. However, I've seen no complaints about the current
> behavior of the command-line and the KIP doesn't change it at all. So I'd
> rather address things separately.
>
> Gwen
>
> On Fri, May 26, 2017 at 8:10 PM Vahid S Hashemian <
> vahidhashem...@us.ibm.com>
> wrote:
>
> > Gwen, thanks for the KIP.
> > It looks good to me.
> >
> > Just a minor suggestion: It would be great if the command asks for a
> > confirmation (y/n) before deleting the topic (similar to how removing
> ACLs
> > works).
> >
> > Thanks.
> > --Vahid
> >
> >
> >
> > From:   Gwen Shapira <g...@confluent.io>
> > To: "dev@kafka.apache.org" <dev@kafka.apache.org>, Users
> > <us...@kafka.apache.org>
> > Date:   05/26/2017 07:04 AM
> > Subject:    KIP-162: Enable topic deletion by default
> >
> >
> >
> > 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
> >
> >
> >
> >
> >
>

Re: KIP-162: Enable topic deletion by default

[Damian Guy]

+1

On Wed, 31 May 2017 at 13:36 Jim Jagielski <j...@jagunet.com> wrote:

> +1
> > On May 27, 2017, at 9:27 PM, Vahid S Hashemian <
> vahidhashem...@us.ibm.com> wrote:
> >
> > Sure, that sounds good.
> >
> > I suggested that to keep command line behavior consistent.
> > Plus, removal of ACL access is something that can be easily undone, but
> > topic deletion is not reversible.
> > So, perhaps a new follow-up JIRA to this KIP to add the confirmation for
> > topic deletion.
> >
> > Thanks.
> > --Vahid
> >
> >
> >
> > From:   Gwen Shapira <g...@confluent.io>
> > To:     dev@kafka.apache.org, us...@kafka.apache.org
> > Date:   05/27/2017 11:04 AM
> > Subject:Re: KIP-162: Enable topic deletion by default
> >
> >
> >
> > Thanks Vahid,
> >
> > Do you mind if we leave the command-line out of scope for this?
> >
> > I can see why adding confirmations, options to bypass confirmations, etc
> > would be an improvement. However, I've seen no complaints about the
> > current
> > behavior of the command-line and the KIP doesn't change it at all. So I'd
> > rather address things separately.
> >
> > Gwen
> >
> > On Fri, May 26, 2017 at 8:10 PM Vahid S Hashemian
> > <vahidhashem...@us.ibm.com>
> > wrote:
> >
> >> Gwen, thanks for the KIP.
> >> It looks good to me.
> >>
> >> Just a minor suggestion: It would be great if the command asks for a
> >> confirmation (y/n) before deleting the topic (similar to how removing
> > ACLs
> >> works).
> >>
> >> Thanks.
> >> --Vahid
> >>
> >>
> >>
> >> From:   Gwen Shapira <g...@confluent.io>
> >> To: "dev@kafka.apache.org" <dev@kafka.apache.org>, Users
> >> <us...@kafka.apache.org>
> >> Date:   05/26/2017 07:04 AM
> >> Subject:KIP-162: Enable topic deletion by default
> >>
> >>
> >>
> >> 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
> >>
> >>
> >>
> >>
> >>
> >
> >
> >
> >
>
>

Re: [VOTE] KIP-162: Enable topic deletion by default

[Damian Guy]

+1

On Tue, 6 Jun 2017 at 18:19 BigData dev <bigdatadev...@gmail.com> wrote:

> +1 (non-binding)
>
> Thanks,
> Bharat
>
> On Tue, Jun 6, 2017 at 9:21 AM, Ashwin Sinha <ashwin.si...@go-jek.com>
> wrote:
>
> > +1
> >
> > On Tue, Jun 6, 2017 at 11:20 PM, Mickael Maison <
> mickael.mai...@gmail.com>
> > wrote:
> >
> > > +1 (non binding), thanks
> > >
> > > On Tue, Jun 6, 2017 at 2:16 PM, Bill Bejeck <bbej...@gmail.com> wrote:
> > > > +1
> > > >
> > > > -Bill
> > > >
> > > > On Tue, Jun 6, 2017 at 9:08 AM, Ismael Juma <ism...@juma.me.uk>
> wrote:
> > > >
> > > >> Thanks for the KIP, Gwen. +1 (binding).
> > > >>
> > > >> Ismael
> > > >>
> > > >> On Tue, Jun 6, 2017 at 5:37 AM, Gwen Shapira <g...@confluent.io>
> > wrote:
> > > >>
> > > >> > Hi,
> > > >> >
> > > >> > The discussion has been quite positive, so I posted a JIRA, a PR
> and
> > > >> > updated the KIP with the latest decisions.
> > > >> >
> > > >> > Lets officially vote on the KIP:
> > > >> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > >> > 162+-+Enable+topic+deletion+by+default
> > > >> >
> > > >> > JIRA is here: https://issues.apache.org/jira/browse/KAFKA-5384
> > > >> >
> > > >> > Gwen
> > > >> >
> > > >>
> > >
> >
> >
> >
> > --
> > Thanks and Regards,
> > Ashwin
> >
>

Re: [VOTE] KIP-162: Enable topic deletion by default

[Rajini Sivaram]

Thanks for the KIP.

+1 (binding)

On Tue, Jun 6, 2017 at 6:19 PM, BigData dev <bigdatadev...@gmail.com> wrote:

> +1 (non-binding)
>
> Thanks,
> Bharat
>
> On Tue, Jun 6, 2017 at 9:21 AM, Ashwin Sinha <ashwin.si...@go-jek.com>
> wrote:
>
> > +1
> >
> > On Tue, Jun 6, 2017 at 11:20 PM, Mickael Maison <
> mickael.mai...@gmail.com>
> > wrote:
> >
> > > +1 (non binding), thanks
> > >
> > > On Tue, Jun 6, 2017 at 2:16 PM, Bill Bejeck <bbej...@gmail.com> wrote:
> > > > +1
> > > >
> > > > -Bill
> > > >
> > > > On Tue, Jun 6, 2017 at 9:08 AM, Ismael Juma <ism...@juma.me.uk>
> wrote:
> > > >
> > > >> Thanks for the KIP, Gwen. +1 (binding).
> > > >>
> > > >> Ismael
> > > >>
> > > >> On Tue, Jun 6, 2017 at 5:37 AM, Gwen Shapira <g...@confluent.io>
> > wrote:
> > > >>
> > > >> > Hi,
> > > >> >
> > > >> > The discussion has been quite positive, so I posted a JIRA, a PR
> and
> > > >> > updated the KIP with the latest decisions.
> > > >> >
> > > >> > Lets officially vote on the KIP:
> > > >> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > >> > 162+-+Enable+topic+deletion+by+default
> > > >> >
> > > >> > JIRA is here: https://issues.apache.org/jira/browse/KAFKA-5384
> > > >> >
> > > >> > Gwen
> > > >> >
> > > >>
> > >
> >
> >
> >
> > --
> > Thanks and Regards,
> > Ashwin
> >
>

[jira] [Resolved] (KAFKA-5384) KIP-162: Enable topic deletion by default

[Gwen Shapira (JIRA)]

 [ 
https://issues.apache.org/jira/browse/KAFKA-5384?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Gwen Shapira resolved KAFKA-5384.
-
   Resolution: Fixed
Fix Version/s: (was: 0.12.0.0)
   0.11.1.0

Issue resolved by pull request 3241
[https://github.com/apache/kafka/pull/3241]

> KIP-162: Enable topic deletion by default
> -
>
> Key: KAFKA-5384
> URL: https://issues.apache.org/jira/browse/KAFKA-5384
> Project: Kafka
>  Issue Type: Improvement
>  Components: core
>Reporter: Gwen Shapira
> Fix For: 0.11.1.0
>
>
> Change default of delete.topic.enable to true
> Remove delete.topic.enable config from config/server.properties.
> See KIP for details: 
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-162+-+Enable+topic+deletion+by+default



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Re: [VOTE] KIP-162: Enable topic deletion by default

[BigData dev]

+1 (non-binding)

Thanks,
Bharat

On Tue, Jun 6, 2017 at 9:21 AM, Ashwin Sinha <ashwin.si...@go-jek.com>
wrote:

> +1
>
> On Tue, Jun 6, 2017 at 11:20 PM, Mickael Maison <mickael.mai...@gmail.com>
> wrote:
>
> > +1 (non binding), thanks
> >
> > On Tue, Jun 6, 2017 at 2:16 PM, Bill Bejeck <bbej...@gmail.com> wrote:
> > > +1
> > >
> > > -Bill
> > >
> > > On Tue, Jun 6, 2017 at 9:08 AM, Ismael Juma <ism...@juma.me.uk> wrote:
> > >
> > >> Thanks for the KIP, Gwen. +1 (binding).
> > >>
> > >> Ismael
> > >>
> > >> On Tue, Jun 6, 2017 at 5:37 AM, Gwen Shapira <g...@confluent.io>
> wrote:
> > >>
> > >> > Hi,
> > >> >
> > >> > The discussion has been quite positive, so I posted a JIRA, a PR and
> > >> > updated the KIP with the latest decisions.
> > >> >
> > >> > Lets officially vote on the KIP:
> > >> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > >> > 162+-+Enable+topic+deletion+by+default
> > >> >
> > >> > JIRA is here: https://issues.apache.org/jira/browse/KAFKA-5384
> > >> >
> > >> > Gwen
> > >> >
> > >>
> >
>
>
>
> --
> Thanks and Regards,
> Ashwin
>

Re: [VOTE] KIP-162: Enable topic deletion by default

[Ashwin Sinha]

+1

On Tue, Jun 6, 2017 at 11:20 PM, Mickael Maison <mickael.mai...@gmail.com>
wrote:

> +1 (non binding), thanks
>
> On Tue, Jun 6, 2017 at 2:16 PM, Bill Bejeck <bbej...@gmail.com> wrote:
> > +1
> >
> > -Bill
> >
> > On Tue, Jun 6, 2017 at 9:08 AM, Ismael Juma <ism...@juma.me.uk> wrote:
> >
> >> Thanks for the KIP, Gwen. +1 (binding).
> >>
> >> Ismael
> >>
> >> On Tue, Jun 6, 2017 at 5:37 AM, Gwen Shapira <g...@confluent.io> wrote:
> >>
> >> > Hi,
> >> >
> >> > The discussion has been quite positive, so I posted a JIRA, a PR and
> >> > updated the KIP with the latest decisions.
> >> >
> >> > Lets officially vote on the KIP:
> >> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> >> > 162+-+Enable+topic+deletion+by+default
> >> >
> >> > JIRA is here: https://issues.apache.org/jira/browse/KAFKA-5384
> >> >
> >> > Gwen
> >> >
> >>
>



-- 
Thanks and Regards,
Ashwin

Re: [VOTE] KIP-162: Enable topic deletion by default

[Mickael Maison]

+1 (non binding), thanks

On Tue, Jun 6, 2017 at 2:16 PM, Bill Bejeck <bbej...@gmail.com> wrote:
> +1
>
> -Bill
>
> On Tue, Jun 6, 2017 at 9:08 AM, Ismael Juma <ism...@juma.me.uk> wrote:
>
>> Thanks for the KIP, Gwen. +1 (binding).
>>
>> Ismael
>>
>> On Tue, Jun 6, 2017 at 5:37 AM, Gwen Shapira <g...@confluent.io> wrote:
>>
>> > Hi,
>> >
>> > The discussion has been quite positive, so I posted a JIRA, a PR and
>> > updated the KIP with the latest decisions.
>> >
>> > Lets officially vote on the KIP:
>> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
>> > 162+-+Enable+topic+deletion+by+default
>> >
>> > JIRA is here: https://issues.apache.org/jira/browse/KAFKA-5384
>> >
>> > Gwen
>> >
>>

Re: [VOTE] KIP-162: Enable topic deletion by default

[Bill Bejeck]

+1

-Bill

On Tue, Jun 6, 2017 at 9:08 AM, Ismael Juma <ism...@juma.me.uk> wrote:

> Thanks for the KIP, Gwen. +1 (binding).
>
> Ismael
>
> On Tue, Jun 6, 2017 at 5:37 AM, Gwen Shapira <g...@confluent.io> wrote:
>
> > Hi,
> >
> > The discussion has been quite positive, so I posted a JIRA, a PR and
> > updated the KIP with the latest decisions.
> >
> > Lets officially vote on the KIP:
> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > 162+-+Enable+topic+deletion+by+default
> >
> > JIRA is here: https://issues.apache.org/jira/browse/KAFKA-5384
> >
> > Gwen
> >
>

Re: [VOTE] KIP-162: Enable topic deletion by default

[Ismael Juma]

Thanks for the KIP, Gwen. +1 (binding).

Ismael

On Tue, Jun 6, 2017 at 5:37 AM, Gwen Shapira <g...@confluent.io> wrote:

> Hi,
>
> The discussion has been quite positive, so I posted a JIRA, a PR and
> updated the KIP with the latest decisions.
>
> Lets officially vote on the KIP:
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> 162+-+Enable+topic+deletion+by+default
>
> JIRA is here: https://issues.apache.org/jira/browse/KAFKA-5384
>
> Gwen
>

Re: [VOTE] KIP-162: Enable topic deletion by default

[Vahid S Hashemian]

+1 (non-binding)

Thanks.
--Vahid



From:   Gwen Shapira <g...@confluent.io>
To: "dev@kafka.apache.org" <dev@kafka.apache.org>, Users 
<us...@kafka.apache.org>
Date:   06/05/2017 09:38 PM
Subject:    [VOTE] KIP-162: Enable topic deletion by default



Hi,

The discussion has been quite positive, so I posted a JIRA, a PR and
updated the KIP with the latest decisions.

Lets officially vote on the KIP:
https://cwiki.apache.org/confluence/display/KAFKA/KIP-162+-+Enable+topic+deletion+by+default


JIRA is here: https://issues.apache.org/jira/browse/KAFKA-5384

Gwen

[VOTE] KIP-162: Enable topic deletion by default

[Gwen Shapira]

Hi,

The discussion has been quite positive, so I posted a JIRA, a PR and
updated the KIP with the latest decisions.

Lets officially vote on the KIP:
https://cwiki.apache.org/confluence/display/KAFKA/KIP-162+-+Enable+topic+deletion+by+default

JIRA is here: https://issues.apache.org/jira/browse/KAFKA-5384

Gwen

[jira] [Commented] (KAFKA-5384) KIP-162: Enable topic deletion by default

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/KAFKA-5384?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16038121#comment-16038121
 ] 

ASF GitHub Bot commented on KAFKA-5384:
---

GitHub user gwenshap opened a pull request:

https://github.com/apache/kafka/pull/3241

KAFKA-5384: Enable topic deletion by default


https://cwiki.apache.org/confluence/display/KAFKA/KIP-162+-+Enable+topic+deletion+by+default

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/gwenshap/kafka KAFKA-5384

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/kafka/pull/3241.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #3241


commit 6aeb369090c0156563e6d7adf42b8d63cc9454bb
Author: Gwen Shapira <csh...@gmail.com>
Date:   2017-06-06T04:33:12Z

KAFKA-5384: Enable topic deletion by default

----


> KIP-162: Enable topic deletion by default
> -
>
> Key: KAFKA-5384
> URL: https://issues.apache.org/jira/browse/KAFKA-5384
> Project: Kafka
>  Issue Type: Improvement
>  Components: core
>Reporter: Gwen Shapira
> Fix For: 0.12.0.0
>
>
> Change default of delete.topic.enable to true
> Remove delete.topic.enable config from config/server.properties.
> See KIP for details: 
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-162+-+Enable+topic+deletion+by+default



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Created] (KAFKA-5384) KIP-162: Enable topic deletion by default

[Gwen Shapira (JIRA)]

Gwen Shapira created KAFKA-5384:
---

 Summary: KIP-162: Enable topic deletion by default
 Key: KAFKA-5384
 URL: https://issues.apache.org/jira/browse/KAFKA-5384
 Project: Kafka
  Issue Type: Improvement
  Components: core
Reporter: Gwen Shapira
 Fix For: 0.12.0.0


Change default of delete.topic.enable to true
Remove delete.topic.enable config from config/server.properties.

See KIP for details: 
https://cwiki.apache.org/confluence/display/KAFKA/KIP-162+-+Enable+topic+deletion+by+default



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Re: KIP-162: Enable topic deletion by default

[Jim Jagielski]

+1
> On May 27, 2017, at 9:27 PM, Vahid S Hashemian <vahidhashem...@us.ibm.com> 
> wrote:
> 
> Sure, that sounds good.
> 
> I suggested that to keep command line behavior consistent.
> Plus, removal of ACL access is something that can be easily undone, but 
> topic deletion is not reversible.
> So, perhaps a new follow-up JIRA to this KIP to add the confirmation for 
> topic deletion.
> 
> Thanks.
> --Vahid
> 
> 
> 
> From:   Gwen Shapira <g...@confluent.io>
> To: dev@kafka.apache.org, us...@kafka.apache.org
> Date:   05/27/2017 11:04 AM
> Subject:Re: KIP-162: Enable topic deletion by default
> 
> 
> 
> Thanks Vahid,
> 
> Do you mind if we leave the command-line out of scope for this?
> 
> I can see why adding confirmations, options to bypass confirmations, etc
> would be an improvement. However, I've seen no complaints about the 
> current
> behavior of the command-line and the KIP doesn't change it at all. So I'd
> rather address things separately.
> 
> Gwen
> 
> On Fri, May 26, 2017 at 8:10 PM Vahid S Hashemian 
> <vahidhashem...@us.ibm.com>
> wrote:
> 
>> Gwen, thanks for the KIP.
>> It looks good to me.
>> 
>> Just a minor suggestion: It would be great if the command asks for a
>> confirmation (y/n) before deleting the topic (similar to how removing 
> ACLs
>> works).
>> 
>> Thanks.
>> --Vahid
>> 
>> 
>> 
>> From:   Gwen Shapira <g...@confluent.io>
>> To: "dev@kafka.apache.org" <dev@kafka.apache.org>, Users
>> <us...@kafka.apache.org>
>> Date:   05/26/2017 07:04 AM
>> Subject:KIP-162: Enable topic deletion by default
>> 
>> 
>> 
>> 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
>> 
>> 
>> 
>> 
>> 
> 
> 
> 
> 

Re: KIP-162: Enable topic deletion by default

[Sandeep Nemuri]

+1

On Sun, May 28, 2017 at 6:57 AM, Vahid S Hashemian <
vahidhashem...@us.ibm.com> wrote:

> Sure, that sounds good.
>
> I suggested that to keep command line behavior consistent.
> Plus, removal of ACL access is something that can be easily undone, but
> topic deletion is not reversible.
> So, perhaps a new follow-up JIRA to this KIP to add the confirmation for
> topic deletion.
>
> Thanks.
> --Vahid
>
>
>
> From:   Gwen Shapira <g...@confluent.io>
> To: dev@kafka.apache.org, us...@kafka.apache.org
> Date:   05/27/2017 11:04 AM
> Subject:Re: KIP-162: Enable topic deletion by default
>
>
>
> Thanks Vahid,
>
> Do you mind if we leave the command-line out of scope for this?
>
> I can see why adding confirmations, options to bypass confirmations, etc
> would be an improvement. However, I've seen no complaints about the
> current
> behavior of the command-line and the KIP doesn't change it at all. So I'd
> rather address things separately.
>
> Gwen
>
> On Fri, May 26, 2017 at 8:10 PM Vahid S Hashemian
> <vahidhashem...@us.ibm.com>
> wrote:
>
> > Gwen, thanks for the KIP.
> > It looks good to me.
> >
> > Just a minor suggestion: It would be great if the command asks for a
> > confirmation (y/n) before deleting the topic (similar to how removing
> ACLs
> > works).
> >
> > Thanks.
> > --Vahid
> >
> >
> >
> > From:   Gwen Shapira <g...@confluent.io>
> > To: "dev@kafka.apache.org" <dev@kafka.apache.org>, Users
> > <us...@kafka.apache.org>
> > Date:   05/26/2017 07:04 AM
> > Subject:KIP-162: Enable topic deletion by default
> >
> >
> >
> > 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
> >
> >
> >
> >
> >
>
>
>
>
>


-- 
*  Regards*
*  Sandeep Nemuri*

Re: KIP-162: Enable topic deletion by default

[Vahid S Hashemian]

Sure, that sounds good.

I suggested that to keep command line behavior consistent.
Plus, removal of ACL access is something that can be easily undone, but 
topic deletion is not reversible.
So, perhaps a new follow-up JIRA to this KIP to add the confirmation for 
topic deletion.

Thanks.
--Vahid



From:   Gwen Shapira <g...@confluent.io>
To: dev@kafka.apache.org, us...@kafka.apache.org
Date:   05/27/2017 11:04 AM
Subject:    Re: KIP-162: Enable topic deletion by default



Thanks Vahid,

Do you mind if we leave the command-line out of scope for this?

I can see why adding confirmations, options to bypass confirmations, etc
would be an improvement. However, I've seen no complaints about the 
current
behavior of the command-line and the KIP doesn't change it at all. So I'd
rather address things separately.

Gwen

On Fri, May 26, 2017 at 8:10 PM Vahid S Hashemian 
<vahidhashem...@us.ibm.com>
wrote:

> Gwen, thanks for the KIP.
> It looks good to me.
>
> Just a minor suggestion: It would be great if the command asks for a
> confirmation (y/n) before deleting the topic (similar to how removing 
ACLs
> works).
>
> Thanks.
> --Vahid
>
>
>
> From:   Gwen Shapira <g...@confluent.io>
> To: "dev@kafka.apache.org" <dev@kafka.apache.org>, Users
> <us...@kafka.apache.org>
> Date:   05/26/2017 07:04 AM
> Subject:KIP-162: Enable topic deletion by default
>
>
>
> 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
>
>
>
>
>

Re: KIP-162: Enable topic deletion by default

[Gwen Shapira]

Agreed and updated the KIP accordingly. Thank you!

On Sat, May 27, 2017 at 1:25 AM Guozhang Wang <wangg...@gmail.com> wrote:

> I'd say just remove those two lines.
>
> On Fri, May 26, 2017 at 7:55 AM, Gwen Shapira <g...@confluent.io> wrote:
>
> > 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
> > > > > >
> > > > >
> > > >
> > >
> >
>
>
>
> --
> -- Guozhang
>

Re: KIP-162: Enable topic deletion by default

[Gwen Shapira]

Thanks Vahid,

Do you mind if we leave the command-line out of scope for this?

I can see why adding confirmations, options to bypass confirmations, etc
would be an improvement. However, I've seen no complaints about the current
behavior of the command-line and the KIP doesn't change it at all. So I'd
rather address things separately.

Gwen

On Fri, May 26, 2017 at 8:10 PM Vahid S Hashemian <vahidhashem...@us.ibm.com>
wrote:

> Gwen, thanks for the KIP.
> It looks good to me.
>
> Just a minor suggestion: It would be great if the command asks for a
> confirmation (y/n) before deleting the topic (similar to how removing ACLs
> works).
>
> Thanks.
> --Vahid
>
>
>
> From:   Gwen Shapira <g...@confluent.io>
> To: "dev@kafka.apache.org" <dev@kafka.apache.org>, Users
> <us...@kafka.apache.org>
> Date:   05/26/2017 07:04 AM
> Subject:KIP-162: Enable topic deletion by default
>
>
>
> 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
>
>
>
>
>

Re: KIP-162: Enable topic deletion by default

[Gwen Shapira]

This KIP only changes a configuration default, so the two changes seem
independent to me.

And since both changes will be included in the release following 0.11
(October) and no sooner, I don't know if the order matters that much.

Am I missing anything regarding KAFKA-4893 that means KIP-162 isn't as
simple as I assume?

Gwen

On Sat, May 27, 2017 at 4:25 AM Onur Karaman <onurkaraman.apa...@gmail.com>
wrote:

> Would it make sense to resolve KAFKA-4893 before enabling it by default, as
> fixing the ticket would likely involve changing the log directory
> structure?
>
> On Fri, May 26, 2017 at 3:24 PM, Guozhang Wang <wangg...@gmail.com> wrote:
>
> > I'd say just remove those two lines.
> >
> > On Fri, May 26, 2017 at 7:55 AM, Gwen Shapira <g...@confluent.io> wrote:
> >
> > > 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
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> >
> >
> > --
> > -- Guozhang
> >
>

Re: KIP-162: Enable topic deletion by default

[Onur Karaman]

Would it make sense to resolve KAFKA-4893 before enabling it by default, as
fixing the ticket would likely involve changing the log directory structure?

On Fri, May 26, 2017 at 3:24 PM, Guozhang Wang <wangg...@gmail.com> wrote:

> I'd say just remove those two lines.
>
> On Fri, May 26, 2017 at 7:55 AM, Gwen Shapira <g...@confluent.io> wrote:
>
> > 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
> > > > > >
> > > > >
> > > >
> > >
> >
>
>
>
> --
> -- Guozhang
>

Re: KIP-162: Enable topic deletion by default

[Guozhang Wang]

I'd say just remove those two lines.

On Fri, May 26, 2017 at 7:55 AM, Gwen Shapira <g...@confluent.io> wrote:

> 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
> > > > >
> > > >
> > >
> >
>



-- 
-- Guozhang

Re: KIP-162: Enable topic deletion by default

[Jim Jagielski]

> On May 26, 2017, at 1:10 PM, Vahid S Hashemian  
> wrote:
> 
> Gwen, thanks for the KIP.
> It looks good to me.
> 
> Just a minor suggestion: It would be great if the command asks for a 
> confirmation (y/n) before deleting the topic (similar to how removing ACLs 
> works).
> 

+1 (or some sort of 'force' option to bypass the
query)

Re: KIP-162: Enable topic deletion by default

[Vahid S Hashemian]

Gwen, thanks for the KIP.
It looks good to me.

Just a minor suggestion: It would be great if the command asks for a 
confirmation (y/n) before deleting the topic (similar to how removing ACLs 
works).

Thanks.
--Vahid



From:   Gwen Shapira <g...@confluent.io>
To: "dev@kafka.apache.org" <dev@kafka.apache.org>, Users 
<us...@kafka.apache.org>
Date:   05/26/2017 07:04 AM
Subject:    KIP-162: Enable topic deletion by default



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

Re: KIP-162: Enable topic deletion by default

[Bill Bejeck]

+1

On Fri, May 26, 2017 at 10:55 AM, Gwen Shapira <g...@confluent.io> wrote:

> 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
> > > > >
> > > >
> > >
> >
>

Re: KIP-162: Enable topic deletion by default

[Gwen Shapira]

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
> > > >
> > >
> >
>

Re: KIP-162: Enable topic deletion by default

[Tom Crayford]

+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
> > >
> >
>

Re: KIP-162: Enable topic deletion by default

[Damian Guy]

+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
> >
>

Re: KIP-162: Enable topic deletion by default

[Ismael Juma]

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
>

Re: KIP-162: Enable topic deletion by default

[Jorge Esteban Quilcate Otoya]

+1

El vie., 26 may. 2017 a las 16:14, Matthias J. Sax (<matth...@confluent.io>)
escribió:

> +1
>
> On 5/26/17 7:03 AM, Gwen Shapira 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
> >
>
>

Re: KIP-162: Enable topic deletion by default

[Matthias J. Sax]

+1

On 5/26/17 7:03 AM, Gwen Shapira 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
> 



signature.asc
Description: OpenPGP digital signature

KIP-162: Enable topic deletion by default

[Gwen Shapira]

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