Re: [DISCUSS] KIP-75 - Add per-connector Converters

2016-08-14 Thread Neha Narkhede
Makes sense to me. +1 On Fri, Aug 12, 2016 at 7:43 PM Sriram Subramanian wrote: > +1 > > On Fri, Aug 12, 2016 at 7:36 PM, Gwen Shapira wrote: > > > Looks good to me too - simple, usable and the use-case is clear. > > > > Gwen > > > > On Fri, Aug 12, 2016 at 12:56 PM, Ewen Cheslack-Postava > >

Re: [DISCUSS] KIP-75 - Add per-connector Converters

2016-08-12 Thread Sriram Subramanian
+1 On Fri, Aug 12, 2016 at 7:36 PM, Gwen Shapira wrote: > Looks good to me too - simple, usable and the use-case is clear. > > Gwen > > On Fri, Aug 12, 2016 at 12:56 PM, Ewen Cheslack-Postava > wrote: > > Hi all, > > > > I've added KIP-75 - Add per-connector Converters > > > > https://cwiki.apa

Re: [DISCUSS] KIP-75 - Add per-connector Converters

2016-08-12 Thread Gwen Shapira
Looks good to me too - simple, usable and the use-case is clear. Gwen On Fri, Aug 12, 2016 at 12:56 PM, Ewen Cheslack-Postava wrote: > Hi all, > > I've added KIP-75 - Add per-connector Converters > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-75+-+Add+per-connector+Converters > > This

Re: [DISCUSS] KIP-75 - Add per-connector Converters

2016-08-12 Thread Ismael Juma
Sounds sensible to me. Ismael On Fri, Aug 12, 2016 at 8:56 PM, Ewen Cheslack-Postava wrote: > Hi all, > > I've added KIP-75 - Add per-connector Converters > > https://cwiki.apache.org/confluence/display/KAFKA/KIP- > 75+-+Add+per-connector+Converters > > This is a very small completely backwards

[DISCUSS] KIP-75 - Add per-connector Converters

2016-08-12 Thread Ewen Cheslack-Postava
Hi all, I've added KIP-75 - Add per-connector Converters https://cwiki.apache.org/confluence/display/KAFKA/KIP-75+-+Add+per-connector+Converters This is a very small completely backwards compatible change, a patch is already available https://github.com/apache/kafka/pull/1721, and I think probab