Hi Ted, Thank you for reviewing the KIP.
*Would decreasing network/IO threads be supported ?* Yes, As described in the KIP, some connections will be closed if network thread count is reduced (and reconnections will be processed on remaining threads) *What if some keys in configs are not in the Set returned by reconfigurableConfigs()? Would exception be thrown ?* No, *reconfigurableConfigs() *will be used to decide which classes are notified when a configuration update is made*. **reconfigure(Map<String, ?> configs)* will be invoked with all of the configured configs of the broker, similar to *configure(Map<String, ?> configs). *For example, when *SslChannelBuilder* is made reconfigurable, it could just create a new SslFactory with the latest configs, using the same code as *configure()*. We avoid reconfiguring *SslChannelBuilder *unnecessarily*, *for example if a topic config has changed, since topic configs are not listed in the *SslChannelBuilder#**reconfigurableConfigs().* *The sample commands for bin/kafka-configs include '--add-config'. Would there be '--remove-config' ?* bin/kafka-configs.sh is an existing tool whose parameters will not be modified by this KIP. There is a --delete-config option. *ssl.keystore.password appears a few lines above. Would there be any issue with mixture of connections (with old and new password) ?* No, passwords (and the actual keystore) are only used during authentication. Any channel created using the old SslFactory will not be impacted. Regards, Rajini On Mon, Nov 20, 2017 at 4:39 PM, Ted Yu <yuzhih...@gmail.com> wrote: > bq. (e.g. increase network/IO threads) > > Would decreasing network/IO threads be supported ? > > bq. void reconfigure(Map<String, ?> configs); > > What if some keys in configs are not in the Set returned by > reconfigurableConfigs() > ? Would exception be thrown ? > If so, please specify which exception would be thrown. > > The sample commands for bin/kafka-configs include '--add-config'. > Would there be '--remove-config' ? > > bq. Existing connections will not be affected, new connections will use the > new keystore. > > ssl.keystore.password appears a few lines above. Would there be any issue > with mixture of connections (with old and new password) ? > > > Cheers > > > > On Mon, Nov 20, 2017 at 5:57 AM, Rajini Sivaram <rajinisiva...@gmail.com> > wrote: > > > Hi all, > > > > I have submitted KIP-226 to enable dynamic reconfiguration of brokers > > without restart: > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > 226+-+Dynamic+Broker+Configuration > > > > The KIP proposes to extend the current dynamic replication quota > > configuration for brokers to support dynamic reconfiguration of a limited > > set of configuration options that are typically updated during the > lifetime > > of a broker. > > > > Feedback and suggestions are welcome. > > > > Thank you... > > > > Regards, > > > > Rajini > > >
