lianetm commented on PR #14323: URL: https://github.com/apache/kafka/pull/14323#issuecomment-1714511273
Thanks for the comments @dajac, all addressed. There is still a lot to think about regarding how the client will handle the assignments it receives. In order to unblock the integrations that require this state components, I'm leaving here the initial approach for handling assignments, to continue working on it in a follow-up PR if that's OK. So for now this follows the initial idea that the member may continue to receive assignments while still processing one (so uses current, target and nextTarget with the latest). I updated the transitions and added tests for it, but I expect it will change as we continue to work on it (If we ever move towards the idea of the server not sending new assignments until acked this would be simplified, it not it would probably stay similar to how it is now, requiring more thought on the client side to properly handle all the tricky scenarios we've discussed). -- 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]
