[GitHub] [pulsar] lhotari commented on issue #11521: NPE in managed ledger on read failed

[GitBox <gi...@apache.org>]

lhotari commented on issue #11521:
URL: https://github.com/apache/pulsar/issues/11521#issuecomment-895969683


   > For this issue, on a given OpReadEntry object one of readEntriesFailed() and checkReadCompletion() needs to get invoked more than once. From reading through the code, I am not entirely sure how that is possible. What makes it harder is that I don't have a way to reproduce the problem.
   
   @bharanic-dev OpReadEntry isn't thread safe. Breaking thread safety rules can lead to very surprising behaviour which is "impossible" from reading the code. One such example of an "impossible" problem in the past is #10433 . 
   #11387 will fix some issues since most access to OpReadEntry will happen from a single thread. That's why it could help in this case too.


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