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

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/04/20 09:40:02 UTC

[GitHub] [pulsar] lhotari commented on pull request #10280: Fix thread safety issue in ManagedLedgerImpl

lhotari commented on pull request #10280:
URL: https://github.com/apache/pulsar/pull/10280#issuecomment-823135530


   Although it's a clear violation of JLS 17.7, it might not be an issue in reality when running on 64-bit HotSpot based JVM: https://stackoverflow.com/a/25176428 . However it's unclear if the problem could occur when the JVM is running in interpreted mode (before JIT).  I guess it's risky to violate JLS 17.7 in any case?
   


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

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