https://bz.apache.org/bugzilla/show_bug.cgi?id=65333

Mark Thomas <ma...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #1 from Mark Thomas <ma...@apache.org> ---
Thanks for taking the time to report this.

The code prior to the changes had a lot of concurrency issues and the clean-up
was necessary to allow the Manager app to deploy multiple apps in parallel
(think automated tooling deployed several apps at once). The changes to
check(String) were a consequence of fixing the concurrency issues.

I've taken a look at the code. check(String) isn't used internally by Tomcat so
that gives us a little flexibility but I don't see a way of restoring the
previous behaviour without adding a fair amount of code and associated
complexity. Therefore, I have added a note about the change of behaviour to the
migration guides for 8.5.x, 9.0.x and 10.0.x

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to