kafka-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From John Roesler <j...@confluent.io>
Subject Re: [DISCUSS] KIP-331 Add default implementation to close() and configure() for Serializer, Deserializer and Serde
Date Tue, 03 Jul 2018 18:17:28 GMT
Hi again, Chia-Ping,

Thanks! I took a look at it, and it seems like a good change to me.

I don't know if it will generate much discussion, so I recommend just
letting it steep for another day or two and then moving on to a vote if no
one else chimes in.

Just really scraping my mind for concerns to investigate... This change
won't break source compatibility, but will it affect binary compatibility?
For example, if I compile my application against Kafka 2.0, for example,
and then swap in the Kafka jar containing your change on my classpath at
run time, will it still work?

I think it should be binary compatible, assuming Oracle didn't do anything
crazy, since all the method references would be the same. It might we worth
an experiment, though.

Thanks,
-John

On Mon, Jul 2, 2018 at 9:29 PM Chia-Ping Tsai <chia7712@apache.org> wrote:

> > Can you provide a link, please?
>
> Pardon me. I put the page in the incorrect location.
>
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-331+Add+default+implementation+to+close%28%29+and+configure%28%29+for+Serializer%2C+Deserializer+and+Serde
>
> Cheers,
> Chia-Ping
>
> On 2018/07/02 19:45:19, John Roesler <john@confluent.io> wrote:
> > Hi Chia-Ping,
> >
> > I couldn't find KIP-331 in the list of KIPs (
> >
> https://cwiki.apache.org/confluence/display/KAFKA/Kafka+Improvement+Proposals
> > ).
> >
> > Can you provide a link, please?
> >
> > Thanks,
> > -John
> >
> > On Sun, Jul 1, 2018 at 11:33 AM Chia-Ping Tsai <chia7712@apache.org>
> wrote:
> >
> > > hi folks,
> > >
> > > KIP-331 is waiting for any suggestions, feedback and reviews. The main
> > > purpose of the KIP-331 is to add empty implementations to the close()
> and
> > > configure() so as to user can write less code to develop custom
> Serialzier,
> > > Deserializer and Serde.
> > >
> > > Cheers,
> > > Chia-Ping
> > >
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message