gharris1727 commented on PR #13178:
URL: https://github.com/apache/kafka/pull/13178#issuecomment-1423176443

   > I notice that we don't do a read-to-end of the offset syncs topic in 
MirrorCheckpointTask before we begin syncing consumer group offsets, and we 
begin reading that topic from the beginning. This may cause us to sync offsets 
based on stale checkpoints if there are later checkpoints available in the 
topic that we haven't consumed yet. Do you think it might make sense to add a 
read-to-end for the offset syncs topic before we begin syncing consumer group 
offsets in the checkpoint connector?
   
   Ouch, yeah that is certainly an issue that gets worse with my change.
   Before the checkpoints would be non-monotonic for transactional/compacted 
topics, and after it's non-monotonic for everyone.
   I think addressing this in a follow-up is a smart idea, this change is 
already messy enough.


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