You are viewing a plain text version of this content. The canonical link for it is here.

Posted to jira@kafka.apache.org by "philipnee (via GitHub)" <gi...@apache.org> on 2023/05/12 22:08:35 UTC

[GitHub] [kafka] philipnee commented on pull request #13490: KAFKA-14875: Implement wakeup

philipnee commented on PR #13490:
URL: https://github.com/apache/kafka/pull/13490#issuecomment-1546362095

   Thanks, @hachikuji - I addressed your comment.  I took a glance at the code again, I realized there's no lock in place.  Giving it a second thought, I think we might not need it as of now. I think if two threads invoke commitSync(), one will just get blocked by the other until completion.  Is my understanding right there?


-- 
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-unsubscribe@kafka.apache.org

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