Hey John, Thanks for the vote. I added the reason for the rejection of the alternatives. The first one is basically an option to broadcast to all partitions which I felt was restrictive. Instead the KIP allows multicasting to 0-N partitions based upon the partitioner implementation.
Thanks! Sagar. On Sat, Aug 13, 2022 at 7:35 AM John Roesler <vvcep...@apache.org> wrote: > Thanks, Sagar! > > I’m +1 (binding) > > Can you add a short explanation to each rejected alternative? I was > wondering why we wouldn’t provide an overloaded to()/addSink() (the first > rejected alternative), and I had to look back at the Streams code to see > that they both already accept the partitioner (I thought it was a config). > > Thanks! > -John > > On Tue, Aug 9, 2022, at 13:44, Walker Carlson wrote: > > +1 (non binding) > > > > Walker > > > > On Tue, May 31, 2022 at 4:44 AM Sagar <sagarmeansoc...@gmail.com> wrote: > > > >> Hi All, > >> > >> I would like to start a voting thread on > >> > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=211883356 > >> . > >> > >> I am just starting this as the discussion thread has been open for 10+ > >> days. In case there are some comments, we can always discuss them over > >> there. > >> > >> Thanks! > >> Sagar. > >> >
