AntonRoskvist commented on PR #4705:
URL: 
https://github.com/apache/activemq-artemis/pull/4705#issuecomment-1908314543

   @jbertram 
   
   You are correct. I actually tried this at first but got issues with 
deadlocking because of calls to addBinding() from `PostOffice#onNotification`, 
then I did not try it again after having those calls go directly to the 
`addressManager`. Thanks!
   
   During testing of this though, I saw the problem yet again... it's much less 
of an issue after the synchronization but can still happen apparently. I 
narrowed it down to varying delays in `PostOffice#processRoute()`... don't know 
if my fix for that is okay though but I figured if all references are 
nonDurables the call to `storageManager.afterCompleteOperations()` would not be 
necessary, is that right? Have not seen the issue since at least.


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

Reply via email to