+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:     d...@kafka.apache.org, users@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:     "d...@kafka.apache.org" <d...@kafka.apache.org>, Users
> > <users@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*

Reply via email to