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

   This PR is build on top of https://github.com/apache/kafka/pull/13637 and 
https://github.com/apache/kafka/pull/13638.
    
   This PR adds the GroupMetadataManager to the group-coordinator module. This 
manager is responsible for handling the groups management, the members 
management and the entire reconciliation process. At this point, only the new 
consumer group type/protocol is implemented.
   
   As you will see, the new manager is based on an architecture inspired from 
the quorum controller. A request can access/read the state but can't mutate it 
directly. Instead, a list of records is generated together with the response 
and those records will be applied to the state by the runtime framework. We use 
timeline data structures. Note that the runtime framework is not part of this 
PR. It will come in a following one.
   
   For the reviewers, I suggest starting from the 
GroupMetadataManager.consumerGroupHeartbeat method. From there, you will how 
the consumer group heartbeat is handled and how all the classes fit together. 
Then, it is possible to review the classes independently, I suppose.
   
   ### 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: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to