kafka-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Maulin Vasavada <maulin.vasav...@gmail.com>
Subject Re: [DISCUSS] KIP-519: Make SSL context/engine configuration extensible
Date Thu, 26 Sep 2019 05:56:25 GMT
Ack. I should be able to get back to this on Friday.

On Mon, Sep 23, 2019 at 10:35 AM Pellerin, Clement <Clement_Pellerin@ibi.com>
wrote:

> When I worked on KIP-383 I was told the way to pass extra arguments to an
> instance is to add extra arguments to configure. I would now suggest we do
> like the KeySerializer. If you look in KafkaProducer, it creates a
> KeySerializer using AbstractConfig. getConfiguredInstance(). Since
> KeySerializer does not extend KeySerializer, AbstractConfig does not call
> configure(Map). KafkaProducer then calls configure() with two arguments.
> This removes the need for the init() method which would be called too late
> after configure(). By the way, the KeySerializer is not the only interface
> that does this.
>
> In summary, SslEngineFactory does not extend Configurable and it has a
> configure() method with more than 1 argument.
>
> The next item is to spec how config.originals() is passed to
> SslChannelBuilder: the KIP needs to explain we will push the choice of
> configs within the switch in ChannelBuilders.create()
>
> Finally, we need to spec which configs are passed to shouldRebuiltFor(). I
> assume these are now originals() instead of values(). We also need to
> specify if all configs are passed or just the reconfigurable ones.
>
>

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