lhotari commented on pull request #14231: URL: https://github.com/apache/pulsar/pull/14231#issuecomment-1036309041
> looks like we don't have to synchronize here because we can accept the incorrect pending messages at a moment? That is risky. It's undefined what the behavior of ArrayDeque would be if it's accessed without proper synchronization. Usually it would be "just" a data consistency issue. In the case of java.util.HashMap, concurrent modifications and access c[ould lead to an infinite loop before Java 8](https://stackoverflow.com/a/44180452). -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
