Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 360445 - [builder] enable parallel resource loading by default
Summary: [builder] enable parallel resource loading by default
Status: NEW
Alias: None
Product: TMF
Classification: Modeling
Component: Xtext (show other bugs)
Version: 2.0.1   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-10 10:36 EDT by Knut Wannheden CLA
Modified: 2014-03-31 18:39 EDT (History)
1 user (show)

See Also:

Attachments
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 Knut Wannheden CLA 2011-10-10 10:36:48 EDT 
With the patch for bug 358714 we now have support for parallel resource loading. This is however not enabled by default, as the implementation would not be able to deal with resources which dynamically load other resources as part of the load phase (see bug 358714 comment #27).

It would be good if we could devise a strategy to deal with this problem and could then enable the parallel resource loading by default.
Comment 1 Knut Wannheden CLA 2011-11-10 14:42:12 EST 
One problem I've encountered with the current ParallelResourceLoader implementation is that the individual loader's ResourceSets will "inherit" the ResourceDescriptionsProvider.NAMED_BUILDER_SCOPE load option without having the corresponding adapter set. This will lead to problems if a resource does try to access the index during loading.
Comment 2 Knut Wannheden CLA 2011-11-10 14:43:47 EST 
A possible solution to deal with resources which load others during the load phase could be to explicitly unload those other resources again before returning the primary loaded resource. This should correctly proxify any references it has to the other resources.