Hello David, Thanks for the KIP! I have read through the proposal and had one minor and one meta comment. But overall it looks good to me!
1) The JoinGroupRequest format does not have any new fields proposed, so we could either clarify that it is listed here but without modifications (only version bumps) or just remove it from the wiki. 2) Could we consider adding a "protocol version" to allow brokers to select the leader with the highest version? This thought is brought up in https://cwiki.apache.org/confluence/display/KAFKA/KIP-429%3A+Kafka+Consumer+Incremental+Rebalance+Protocol#KIP-429:KafkaConsumerIncrementalRebalanceProtocol-LookingintotheFuture:AssignorVersion. I'm fine with keeping this KIP's scope as is, just wondering if you feel comfortable piggy-backing this change as well if we are going to bump up the JoinGroupReq/Response anyways. Guozhang On Wed, Jan 22, 2020 at 9:10 AM Eno Thereska <eno.there...@gmail.com> wrote: > This is awesome! +1 (non binding) > Eno > > On Tue, Jan 21, 2020 at 10:00 PM Gwen Shapira <g...@confluent.io> wrote: > > > > Thank you for the KIP. Awesomely cloud-native improvement :) > > > > +1 (binding) > > > > > > On Tue, Jan 21, 2020, 9:35 AM David Jacot <dja...@confluent.io> wrote: > > > > > Hi all, > > > > > > I would like to start a vote on KIP-559: Make the Kafka Protocol > Friendlier > > > with L7 Proxies. > > > > > > The KIP is here: > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-559%3A+Make+the+Kafka+Protocol+Friendlier+with+L7+Proxies > > > > > > Thanks, > > > David > > > > -- -- Guozhang
