dajac opened a new pull request, #18261:
URL: https://github.com/apache/kafka/pull/18261

   This patch is the first one in a series to improve how coordinator records 
are managed. It focuses on making coordinator records first class citizen in 
the generator.
   * Introduce `coordinator-key` and `coordinator-value` in the schema;
   * Introduce `apiKey` for those. This is done to avoid relying on the version 
to determine the type.
   * It also allows the generator to enforce some rules: the key cannot use 
flexible versions, the key must have a single version `0`, there must be a key 
and a value for a given api key, etc.
   * It generates an enum with all the coordinator record types. This is pretty 
handy in the code.
   
   The patch also updates the group coordinator to use those.
   
   PS: This breaks the old coordinator because it still relies on the version 
to determine the type. I need to update it.
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


-- 
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]

Reply via email to