lianetm commented on PR #18737: URL: https://github.com/apache/kafka/pull/18737#issuecomment-2624857094
Hey @frankvicky , good point (but you mean `seek` and `position` I guess?) Those are the ones, other than `poll`, that can update the positions. Agree we need to consider them too. In the end it's truly only 2 events behind the 3 calls (`SeekUnvalidatedEvent` and `CheckAndUpdatePositionsEvent`), so one option would be just to make sure that when we process those events in the background, the `commitMgr` updates the positions snapshot to commit. With this, we wouldn't need to change anything in the `PollEvent` I expect. Thoughts? -- 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
