Just a couple of minor points that don't really affect the implementation:

* For nulls, let's just mention the underlying serializers already support
this. I'm actually not sure why they should/need to, but given they do,
let's just defer to that implementation.
* I'm not sure where Float and Double converters are actually useful. The
use cases I know for integer serdes is for keys, but floats seem like a bad
choice for keys. These aren't a lot of overhead to build and maintain, but
if we don't know use cases for the specific types, it might be silly to
spend time and effort building and maintaining them.

Otherwise, this seems simple and straightforward. Generally +1 on the


On Thu, May 17, 2018 at 6:04 PM Magesh Nandakumar <mage...@confluent.io>

> Thanks Randall for the KIP. I think it will be super useful and looks
> pretty straightforward to me.
> Thanks
> Magesh
> On Thu, May 17, 2018 at 4:15 PM, Randall Hauch <rha...@gmail.com> wrote:
> > I'd like to start discussion of a very straightforward proposal for
> Connect
> > to add converters for the basic primitive number types: integer, short,
> > long, double, and float. Here is the KIP:
> >
> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > 305%3A+Add+Connect+primitive+number+converters
> >
> > As mentioned in the KIP, I've created a pull request (
> > https://github.com/apache/kafka/pull/5034) for those looking for
> > implementation details.
> >
> > Any feedback is appreciated.
> >
> > Best regards,
> >
> > Randall
> >

Reply via email to