Thanks, folks. I added some discussion about the potential to support new versions of SaslHandshakeRequest through this mechanism. It may be useful down the road.
I added a little more detail about how the BrokerVersions command would work, as well as some sample output from the current patch. I think it will be pretty useful to be able to query broker versions, especially during an upgrade. cheers, Colin On Thu, Dec 8, 2016, at 09:39, Sriram Subramanian wrote: > +1 (binding) > > On Thu, Dec 8, 2016 at 5:42 AM, Ismael Juma <ism...@juma.me.uk> wrote: > > > Thanks for the KIP Colin, +1 (binding) from me. > > > > I agree with Rajini's suggestions. Also, since the command is a public > > interface, it would be good to include details in the KIP. > > > > Ismael > > > > On Wed, Dec 7, 2016 at 5:17 PM, Colin McCabe <cmcc...@apache.org> wrote: > > > > > Hi all, > > > > > > I heard that the VOTE and DISCUSS threads for the KIP-97 discussion > > > appeared to be in the same email thread for some people using gmail. So > > > I'm reposting in hopes of getting a separate email thread this time for > > > those viewers. :) > > > > > > I'd like to start voting on KIP-97 > > > (https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > > 97%3A+Improved+Kafka+Client+RPC+Compatibility+Policy > > > ). > > > > > > The discussion thread can be found here: > > > https://www.mail-archive.com/dev@kafka.apache.org/msg60955.html > > > > > > Thanks for your feedback. > > > > > > best, > > > Colin McCabe > > > > >