clebertsuconic commented on code in PR #6268:
URL: https://github.com/apache/artemis/pull/6268#discussion_r2875783224
##########
artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/connect/AMQPBrokerConnection.java:
##########
@@ -251,7 +276,20 @@ public synchronized void initialize() throws Exception {
}
@Override
- public synchronized void start() throws Exception {
+ public void start() throws Exception {
+
+ if (lockCoordinator != null) {
+ this.active = false;
+ // this needs to be started before the acceptor, hence a lower
priority on start
+ lockCoordinator.onLockAcquired(this::resume, 5);
Review Comment:
I actually changed this to use the DEFAULT_PRIORITY. The thing that needs
priority actually is the acceptor (first to start, last to stop).. it doesn't
matter on other components
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
