gemmellr commented on PR #6202: URL: https://github.com/apache/artemis/pull/6202#issuecomment-3824935000
Garys idea seems reasonable, in some respects it is a bit weird to warn about something not happening yet, when being explicitly instructed not to wait for it to happen. Could possibly drop it to info level if feeling strongly that it should still be noted, though its not clear why _this_ bit would be noted when it seems the existing usage of that config wasnt causing any warnings. That said, I also dont actually know that we really want this set to 0 all the time in the test suite? Until yesterday the related bit previously waited for as long as needed during the entire test suite, and now because of reusing an existing configuration value it will never wait at all. Feels a bit weird. -- 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]
