Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 331970 - Performance: persist and restore EAR Libraries classpath container
Summary: Performance: persist and restore EAR Libraries classpath container
Status: RESOLVED FIXED
Alias: None
Product: WTP Java EE Tools
Classification: WebTools
Component: jst.j2ee (show other bugs)
Version: 3.2   Edit
Hardware: PC Windows Server 2003
: P3 normal (vote)
Target Milestone: 3.2.3   Edit
Assignee: Jason Sholl CLA
QA Contact: Chuck Bridgham CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-06 20:09 EST by Jason Sholl CLA
Modified: 2010-12-15 14:21 EST (History)
1 user (show)

See Also:
cbridgha: review+
jsholl: review? (ccc)

Attachments
patch (21.17 KB, patch)
2010-12-06 20:11 EST, Jason Sholl CLA 		no flags Details | Diff
patch v2 (37.58 KB, patch)
2010-12-13 17:59 EST, Jason Sholl CLA 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jason Sholl CLA 2010-12-06 20:09:54 EST 
This is the follow on work mentioned in bug 330984 for the EAR Libraries classpath container.  It also contains an improvement to the original bug for the IDependencyGraph.

The design is similar.  Each EAR Libraries classpath container is cached based by project name and is loaded during startup from the cache rather than recomputing.  This shortens the time spent in the critical section of JDT startup.  If a container is restored then a separate job is scheduled to verify and fix the container if necessary, but because this work is farmed off to another job initial startup is not affected.
Comment 1 Jason Sholl CLA 2010-12-06 20:11:35 EST 
Created attachment 184681 [details]
patch
Comment 2 Jason Sholl CLA 2010-12-13 17:59:11 EST 
Created attachment 185100 [details]
patch v2

This patch also contains a similar improvement for the Web App Libraries classpath container.
Comment 3 Chuck Bridgham CLA 2010-12-14 17:13:34 EST 
approve
Comment 4 Jason Sholl CLA 2010-12-15 14:21:59 EST 
Code checked into 32M and HEAD for WTP 3.2.3 and 3.3