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

Mark Thomas <[email protected]> changed:

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

--- Comment #14 from Mark Thomas <[email protected]> 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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to