cshannon commented on PR #2553: URL: https://github.com/apache/activemq/pull/2553#issuecomment-5603311824
> @cshannon I'm fine to do that but not sure it changes anything. > > I've tested the `BrokerService`: > > 1. `slave` defaults to `true` and is reset to `true` on every stop. > 2. It only flips to `false` inside `startAllConnectors()`. > 3. `adminView.setBroker(managedBroker)` (this call makes `BrokerView.broker` non-null) happens in `doStartBroker()` which runs before `startAllConnectors()`. > > So within a single `BrokerService` instance, `broker == null` implies `isSlave() == true` at every point at startup (there's no case where `broker` is null but `isSlave()` has already gone false. Alright, based on that it's probably no point to add the check. As you stated it doesn't really buy us anything. -- 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] For further information, visit: https://activemq.apache.org/contact
