kowshik edited a comment on pull request #9412: URL: https://github.com/apache/kafka/pull/9412#issuecomment-707241757
@abbccdda Sorry for the confusion. Upon second look, I feel we can abandon this PR containing the fix. I was looking at the code again, and it seems the requirement is already take care of in the earlier PRs. If IBP >= `KAFKA_2_7_IV0` we set Broker version to 5, which eventually causes features to be advertised in `BrokerIdZNode.encode`. As a result, the logic is already restrictive like the way we would have liked. Here are some pointers to the code: https://github.com/apache/kafka/blob/fb4f297207ef62f71e4a6d2d0dac75752933043d/core/src/main/scala/kafka/zk/ZkData.scala#L99-L100 https://github.com/apache/kafka/blob/fb4f297207ef62f71e4a6d2d0dac75752933043d/core/src/main/scala/kafka/zk/ZkData.scala#L151-L153 ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
