cmccabe commented on PR #12294: URL: https://github.com/apache/kafka/pull/12294#issuecomment-1233178487
Thanks for finding this, @dengziming . It is a great find and definitely a 3.3 blocker. Two comments: * ZK brokers should not expose metadata.version (they should have a supported range from 0-0, aka no supported range) * I want to get rid of all the "use an empty feature set" factory functions in `ApiVersionsResponse.java`. I made this comment before on a different review, and we left them in, and now we had this major bug. It's time to get rid of all of them. If people want an empty feature set, they can specify that explicitly. Thanks again for the PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
