Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 312984 - [repository] Composite repo should have the ability to be marked in error if a child repo is bad
Summary: [repository] Composite repo should have the ability to be marked in error if ...
Status: RESOLVED FIXED
Alias: None
Product: Equinox
Classification: Eclipse Project
Component: p2 (show other bugs)
Version: 3.6   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 normal (vote)
Target Milestone: 3.7 M7   Edit
Assignee: Katya Stoycheva CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-14 17:39 EDT by Pascal Rapicault CLA
Modified: 2011-04-21 14:27 EDT (History)
0 users

See Also:

Attachments
Composite repository atomic loading support (68.93 KB, patch)
2011-04-21 12:52 EDT, Katya Stoycheva CLA 		no flags Details | Diff
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 Pascal Rapicault CLA 2010-05-14 17:39:11 EDT 
In some cases the composite repo really needs to have all its children enabled and properly loaded to be useful. In other cases this is not necessary.

Composite repo should have a flag to indicate which behaviour they like.

In the failing mode the error should be reported to the user on the composite repo.
Comment 1 Katya Stoycheva CLA 2011-04-21 12:52:27 EDT 
Created attachment 193860 [details]
Composite repository atomic loading support

Composite repository (both Artifact and Metadata) now have a flag "p2.atomic.composite.loading" to indicate whether a failure while loading any of its children should result in failure in composite repository loading.
The default value is false.
If composite repository fails to load and the flag above is set to true all the children loaded on the behalf of the composite repository will be removed from the repository manager. If some of the children have already been available before loading of the composite repository they won't be removed.
Comment 2 Pascal Rapicault CLA 2011-04-21 14:27:18 EDT 
I've reviewed this patch and applied it. The code is really clean and properly tested. Thx for working on this.