mimaison commented on a change in pull request #10743: URL: https://github.com/apache/kafka/pull/10743#discussion_r657856438
########## File path: clients/src/main/resources/common/message/FindCoordinatorRequest.json ########## @@ -23,12 +23,16 @@ // Version 2 is the same as version 1. // // Version 3 is the first flexible version. - "validVersions": "0-3", + // + // Version 4 adds CoordinatorKeys + "validVersions": "0-4", "flexibleVersions": "3+", "fields": [ - { "name": "Key", "type": "string", "versions": "0+", + { "name": "Key", "type": "string", "versions": "0-3", "about": "The coordinator key." }, { "name": "KeyType", "type": "int8", "versions": "1+", "default": "0", "ignorable": false, Review comment: `KeyType` was added in version 1 to resolve transaction coordinators. Before that, FindCoordinator was only able to resolve group coordinators. It's still used in version 4 as we typically never want to resolve transaction and consumer coordinators at the same time so it determines the type of all keys in the request. -- 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