Hi Jun, Thank you for the review. Yes, we should add rack id to Subscription, had missed that part. Updated the KIP, thank you for pointing that out!
Regards, Rajini On Wed, Nov 2, 2022 at 7:06 PM Jun Rao <j...@confluent.io.invalid> wrote: > Hi, Rajini, > > Thanks for the KIP. Just one comment. > > Should we add rackId to GroupSubscription.Subscription for each member? > > Thanks, > > Jun > > On Wed, Nov 2, 2022 at 4:57 AM Rajini Sivaram <rajinisiva...@gmail.com> > wrote: > > > Hi all, > > > > I have submitted KIP-881 to implement rack-aware partition assignment for > > consumers: > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-881%3A+Rack-aware+Partition+Assignment+for+Kafka+Consumers > > . > > It adds rack id to the consumer group protocol to propagate rack > > information so that rack-aware assignors can be added to benefit from > > locality. > > > > Feedback and suggestions are welcome! > > > > Thank you, > > > > Rajini > > >
