[GitHub] [tomcat] ChristopherSchultz commented on pull request #481: changing HashMap to LinkedHashMap to maintain execution order of deployment

[GitBox <gi...@apache.org>]

ChristopherSchultz commented on pull request #481:
URL: https://github.com/apache/tomcat/pull/481#issuecomment-1065380953


   What exactly does this accomplish? LinkedHashMap just produces iterators which maintain insertion-order. It doesn't impose any order on when they were inserted.
   
   Is this intended to maintain _redeployment_-order when auto-deployment is enabled and all applications are updated simultaneously? If so, it's entirely possible that your application updates would be processed across multiple background-processing executions and therefore wouldn't necessarily be redeployed in the order you expected, anyway.
   
   I must not understand the use-case, here.


-- 
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: dev-unsubscribe@tomcat.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org