Hello all, Bumping this thread as we are one binding vote short accepting this KIP. Please let me know if you have any extra concerns and/or suggestions.
Regards, Levani > On 12. Mar 2021, at 13:14, Levani Kokhreidze <levani.co...@gmail.com> wrote: > > Hi Guozhang, > > Thanks for the feedback. I think it makes sense. > I updated the KIP with your proposal [1], it’s a nice optimisation. > I do agree that having the same configuration across Kafka Streams instances > is the reasonable requirement. > > Best, > Levani > > [1] - > https://cwiki.apache.org/confluence/display/KAFKA/KIP-708%3A+Rack+awareness+for+Kafka+Streams > > <https://cwiki.apache.org/confluence/display/KAFKA/KIP-708%3A+Rack+awareness+for+Kafka+Streams> > > >> On 12. Mar 2021, at 03:36, Guozhang Wang <wangg...@gmail.com >> <mailto:wangg...@gmail.com>> wrote: >> >> Hello Levani, >> >> Thanks for the great write-up! I think this proposal makes sense, though I >> have one minor suggestion regarding the protocol format change: note the >> subscription info is part of the group metadata message that we need to >> write into the internal topic, and hence it's always better if we can save >> on the number of bytes written there. For this, I'm wondering if we can >> encode the key part instead of writing raw bytes based on the >> configurations, i.e.: >> >> 1. streams will look at the `task.assignment.rack.awareness` values, and >> encode them in a deterministic manner, e.g. in your example zone = 0, >> cluster = 1. This assumes that all instances will configure this value in >> the same way and then with a deterministic manner all instances will have >> the same encodings, which I think is a reasonable requirement. >> 2. the sent protocol would be "key => short, value => bytes" instead. >> >> >> WDYT? >> >> Otherwise, I'm +1 on the KIP! >> >> Guozhang >> >> >> >> >> On Thu, Mar 11, 2021 at 8:29 AM John Roesler <vvcep...@apache.org >> <mailto:vvcep...@apache.org>> wrote: >> >>> Thanks for the KIP! >>> >>> I'm +1 (binding) >>> >>> -John >>> >>> On Wed, 2021-03-10 at 13:13 +0200, Levani Kokhreidze wrote: >>>> Hello all, >>>> >>>> I’d like to start the voting on KIP-708 [1] >>>> >>>> Best, >>>> Levani >>>> >>>> [1] - >>> https://cwiki.apache.org/confluence/display/KAFKA/KIP-708%3A+Rack+awareness+for+Kafka+Streams >>> >>> <https://cwiki.apache.org/confluence/display/KAFKA/KIP-708%3A+Rack+awareness+for+Kafka+Streams> >>>> >>> >>> >>> >> >> -- >> -- Guozhang >