jbertram opened a new pull request, #5192: URL: https://github.com/apache/activemq-artemis/pull/5192
Before the changes in 15dd24754ae8c6c2dd646fbd214f7a76b33b1197 temporary web resources could proliferate and consume inordinate amounts of disk space because their directory names were generated uniquely every time Jetty was started. However, now that they are deterministic no proliferation is possible. Jetty will create the directories when it starts, remove them when it stops, and if it fails to clean-up on shutdown (e.g. crash from OOME) it will clean-up and recreate them when it starts. Therefore, our own house-keeping of those directories is no longer needed and, in fact, causes problems. For example, when executing the `restartEmbeddedWebServer` management operation the temp web resources will actually be removed inadvertently causing the web console to fail. This commit removes the web temp house-keeping logic as well as the related tests. It also modifies & adds tests to ensure Jetty does this house-keeping on its own. -- 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: gitbox-unsubscr...@activemq.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org For additional commands, e-mail: gitbox-h...@activemq.apache.org For further information, visit: https://activemq.apache.org/contact
