jbertram commented on pull request #3933: URL: https://github.com/apache/activemq-artemis/pull/3933#issuecomment-1030239701
> There still seems to be a lot of tests remaining with such old 'default config' with these being set though... I found mostly smoke tests and examples which were still referencing these config elements. I cleaned those up as well. As for the behavior now that they aren't being set, that shouldn't be a problem. The broker doesn't even use those address setting values anymore. They've simply been ignored since the introduction of their replacements. The change here is really just about presentation. No semantics should be impacted at all. -- 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]
