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 > On 12. Mar 2021, at 03:36, Guozhang Wang <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> 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 >>> >> >> >> > > -- > -- Guozhang