Hi Ashwin,

> I was referring to (and did not understand) the removal of L141 in 
> clients/src/main/java/org/apache/kafka/common/security/ssl/SslFactory.java
This line is moved to "new" private method `instantiateSslEngineFactory0 `. 
Please take a look at the `SslFactory:L132` at the patch.
Just dummy refactoring.

> Yes, I think this class [SslEngineFactory] should be moved to something like 
> `server-common` module - but would like any of the committers to comment on 
> this.
Sorry, not catch an idea.
SslEngineFactory - public interface is placed at the 'clients' project. I don't 
know a more common place

Reply via email to