Hi, Maulin, Thanks for the KIP. +1 from me. Just a couple of minor comments below.
1. Should the package name of the new interface SslEngineFactory be org.apache.kafka.common.security.ssl instead of org.apache.kafka.common.security.auth? 2. Could you document when shouldBeRebuilt() will be called? Jun On Mon, Mar 30, 2020 at 5:07 PM Maulin Vasavada <maulin.vasav...@gmail.com> wrote: > ^^^ bump ^^^ The vote is open for 2-3 days and gotten 1 Binding vote so > far, can you please vote so that we can try to move forward with changes? > > On Thu, Mar 26, 2020 at 4:11 PM Zhou, Thomas <thz...@paypal.com.invalid> > wrote: > > > +1 (non-binding) > > > > Regards, > > Thomas > > > > On 3/26/20, 12:36 PM, "Rajini Sivaram" <rajinisiva...@gmail.com> wrote: > > > > +1 (binding) > > Thanks for the KIP, Maulin! > > > > Regards, > > > > Rajini > > > > On Thu, Mar 26, 2020 at 4:14 PM Maulin Vasavada < > > maulin.vasav...@gmail.com> > > wrote: > > > > > FYI - we have updated the KIP documentation also with appropriate > > code > > > samples for interfaces and few important changes. > > > > > > Thanks > > > Maulin > > > > > > On Wed, Mar 25, 2020 at 10:21 AM Maulin Vasavada < > > > maulin.vasav...@gmail.com> > > > wrote: > > > > > > > bump > > > > > > > > On Wed, Mar 25, 2020 at 10:20 AM Maulin Vasavada < > > > > maulin.vasav...@gmail.com> wrote: > > > > > > > >> Hi all > > > >> > > > >> After much await on the approach conclusion we have a PR > > > >> > > > https://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fkafka%2Fpull%2F8338&data=01%7C01%7Cthzhou%40paypal.com%7C4520b56f3b1f44cceddb08d7d1bd052a%7Cfb00791460204374977e21bac5f3f4c8%7C1&sdata=1ydk0OMaucb8QhTyyQ8Ua3ereGzcS4usRlavU1RixkE%3D&reserved=0 > > . > > > >> > > > >> Can you please provide your vote so that we can more this > forward? > > > >> > > > >> Thanks > > > >> Maulin > > > >> > > > >> On Sun, Jan 26, 2020 at 11:03 PM Maulin Vasavada < > > > >> maulin.vasav...@gmail.com> wrote: > > > >> > > > >>> Hi all > > > >>> > > > >>> After a good discussion on the KIP at > > > >>> > > > https://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.mail-archive.com%2Fdev%40kafka.apache.org%2Fmsg101011.html&data=01%7C01%7Cthzhou%40paypal.com%7C4520b56f3b1f44cceddb08d7d1bd052a%7Cfb00791460204374977e21bac5f3f4c8%7C1&sdata=qsvbqkoxL6NSPDV6rm9B9xqZG5xvYaZkj0cYrTM6bPw%3D&reserved=0 > > I > > > >>> think we are ready to start voting. > > > >>> > > > >>> KIP: > > > >>> > > > > > > https://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fcwiki.apache.org%2Fconfluence%2Fpages%2Fviewpage.action%3FpageId%3D128650952&data=01%7C01%7Cthzhou%40paypal.com%7C4520b56f3b1f44cceddb08d7d1bd052a%7Cfb00791460204374977e21bac5f3f4c8%7C1&sdata=rcqWc2inIbrWlMj2jssHPKcMlHuDuLvicmYHHDYWrF8%3D&reserved=0 > > > >>> > > > >>> The KIP proposes - Making SSLEngine creation pluggable to > support > > > >>> customization of various security related aspects. > > > >>> > > > >>> Thanks > > > >>> Maulin > > > >>> > > > >> > > > > > > > > > >
