https://issues.apache.org/bugzilla/show_bug.cgi?id=46264

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #27758|0                           |1
        is obsolete|                            |

--- Comment #14 from Mark Thomas <ma...@apache.org> 2011-10-11 17:12:24 UTC ---
Created attachment 27761
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=27761
Threaded start, stop and deployment for Contexts

Updated patch that:
- fixes the issues identified by Konstantin
- includes a variation of Felix's patch for context.xml parsing
- fixes a similar issue with web.xml parsing
- fixes an issue that meant the host's executor spun up threads every time it
checked for new apps to deploy

With this patch applied, I see around 30% improvement in start time for 10s of
small, simple applications. This is better, but not the improvement I was
hoping for with 4 threads on an 8-core machine. The bulk of the time appears to
be spent in XML parsing.

I'm continuing to look into to this to see if there is scope for further
improvement. Suggestions and/or additional analysis welcome.

Note: The overhead of creating a digester per app is noticeable at ~5% so I
think it is a price worth paying.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- 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