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 >
