Hi Matthew, Thanks for the update. I'm +1 (binding)
Thank you. Luke On Fri, Feb 11, 2022 at 3:32 PM Matthew de Detrich <matthew.dedetr...@aiven.io.invalid> wrote: > Hi Luke, > > I have just updated the KIP with the changes you requested. > > Regards > > On Fri, Feb 11, 2022 at 4:47 AM Luke Chen <show...@gmail.com> wrote: > > > Hi Matthew, > > > > I checked again the KIP, and it LGTM. > > > > Just a minor comment: > > Maybe add some examples into the KIP to show how users can set both IPv4 > > and IPv6 on the same port. > > And some examples to show how the validation will fail like you listed in > > `Proposed Changes`. > > > > Thank you. > > Luke > > > > > > On Fri, Feb 11, 2022 at 8:54 AM Matthew de Detrich > > <matthew.dedetr...@aiven.io.invalid> wrote: > > > > > Hello everyone > > > > > > I have just updated/rebased the PR against the latest Kafka trunk. Let > me > > > know if anything else is required/missing. > > > > > > Regards > > > > > > On Thu, Jan 13, 2022 at 10:28 AM Matthew de Detrich < > > > matthew.dedetr...@aiven.io> wrote: > > > > > > > Does anyone have any additional comments/regards to help get this PR > > > voted > > > > through? > > > > > > > > On Tue, Nov 23, 2021 at 7:46 AM Josep Prat > <josep.p...@aiven.io.invalid > > > > > > > wrote: > > > > > > > >> Hi Matthew, > > > >> > > > >> Thank you for the PR. > > > >> > > > >> +1 (non binding) from my side. > > > >> > > > >> > > > >> Best, > > > >> > > > >> ——— > > > >> Josep Prat > > > >> > > > >> Aiven Deutschland GmbH > > > >> > > > >> Immanuelkirchstraße 26, 10405 Berlin > > > >> > > > >> Amtsgericht Charlottenburg, HRB 209739 B > > > >> > > > >> Geschäftsführer: Oskari Saarenmaa & Hannu Valtonen > > > >> > > > >> m: +491715557497 > > > >> > > > >> w: aiven.io > > > >> > > > >> e: josep.p...@aiven.io > > > >> > > > >> On Tue, Nov 23, 2021, 07:11 Ivan Yurchenko < > ivan0yurche...@gmail.com> > > > >> wrote: > > > >> > > > >> > Hi, > > > >> > > > > >> > Thank you for the KIP. > > > >> > > > > >> > +1 (non-binding) > > > >> > > > > >> > Ivan > > > >> > > > > >> > > > > >> > On Tue, 23 Nov 2021 at 04:18, Luke Chen <show...@gmail.com> > wrote: > > > >> > > > > >> > > Hi Matthew, > > > >> > > Thanks for the KIP. > > > >> > > It makes sense to allow IPv4 and IPv6 listening on the same port > > for > > > >> the > > > >> > > listener config. > > > >> > > > > > >> > > +1 (non-binding) > > > >> > > > > > >> > > Thank you. > > > >> > > Luke > > > >> > > > > > >> > > On Mon, Nov 22, 2021 at 6:28 PM Matthew de Detrich > > > >> > > <matthew.dedetr...@aiven.io.invalid> wrote: > > > >> > > > > > >> > > > Hello everyone, > > > >> > > > > > > >> > > > I would like to start a vote for KIP-797: Accept duplicate > > > listener > > > >> on > > > >> > > port > > > >> > > > for IPv4/IPv6 > > > >> > > > > > > >> > > > > > > >> > > > > > >> > > > > >> > > > > > > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=195726330 > > > >> > > > > > > >> > > > The purpose of the KIP is to loosen current validation for non > > > >> > advertised > > > >> > > > listeners so that you can have an IPv4 address and an IPv6 > > address > > > >> on > > > >> > the > > > >> > > > same port. All other behaviour remains the same as before > (since > > > >> these > > > >> > > are > > > >> > > > disparate IP stacks there are no technical reasons not to > allow > > > >> this). > > > >> > > > > > > >> > > > PR is located at https://github.com/apache/kafka/pull/11478 > > > >> > > > > > > >> > > > Comments and feedback are welcome! > > > >> > > > > > > >> > > > Regards > > > >> > > > > > > >> > > > -- > > > >> > > > > > > >> > > > Matthew de Detrich > > > >> > > > > > > >> > > > *Aiven Deutschland GmbH* > > > >> > > > > > > >> > > > Immanuelkirchstraße 26, 10405 Berlin > > > >> > > > > > > >> > > > Amtsgericht Charlottenburg, HRB 209739 B > > > >> > > > > > > >> > > > Geschäftsführer: Oskari Saarenmaa & Hannu Valtonen > > > >> > > > > > > >> > > > *m:* +491603708037 > > > >> > > > > > > >> > > > *w:* aiven.io *e:* matthew.dedetr...@aiven.io > > > >> > > > > > > >> > > > > > >> > > > > >> > > > > > > > > > > > > -- > > > > > > > > Matthew de Detrich > > > > > > > > *Aiven Deutschland GmbH* > > > > > > > > Immanuelkirchstraße 26, 10405 Berlin > > > > > > > > Amtsgericht Charlottenburg, HRB 209739 B > > > > > > > > Geschäftsführer: Oskari Saarenmaa & Hannu Valtonen > > > > > > > > *m:* +491603708037 > > > > > > > > *w:* aiven.io *e:* matthew.dedetr...@aiven.io > > > > > > > > > > > > > -- > > > > > > Matthew de Detrich > > > > > > *Aiven Deutschland GmbH* > > > > > > Immanuelkirchstraße 26, 10405 Berlin > > > > > > Amtsgericht Charlottenburg, HRB 209739 B > > > > > > Geschäftsführer: Oskari Saarenmaa & Hannu Valtonen > > > > > > *m:* +491603708037 > > > > > > *w:* aiven.io *e:* matthew.dedetr...@aiven.io > > > > > > > > -- > > Matthew de Detrich > > *Aiven Deutschland GmbH* > > Immanuelkirchstraße 26, 10405 Berlin > > Amtsgericht Charlottenburg, HRB 209739 B > > Geschäftsführer: Oskari Saarenmaa & Hannu Valtonen > > *m:* +491603708037 > > *w:* aiven.io *e:* matthew.dedetr...@aiven.io >
