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

Posted to commits@pulsar.apache.org by "lhotari (via GitHub)" <gi...@apache.org> on 2023/06/05 11:46:52 UTC

[GitHub] [pulsar] lhotari commented on pull request #20488: [fix][broker] Fix race condition in ManagedCursorImpl related to skipping messages

lhotari commented on PR #20488:
URL: https://github.com/apache/pulsar/pull/20488#issuecomment-1576637283

   > I suppose fixes on race condidation requires extra test coverage otherwise it's hard to maintain or prove. Or at least an execution steps graph to show the race and how we fix it.
   
   Yes. However, if you review the code you will notice that the change in this PR makes sense from code analysis and correctness perspective. 


-- 
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: commits-unsubscribe@pulsar.apache.org

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