Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 343020 - [region] need to perform a consistency check when starting from persistent digraph
Summary: [region] need to perform a consistency check when starting from persistent di...
Status: RESOLVED FIXED
Alias: None
Product: Equinox
Classification: Eclipse Project
Component: Components (show other bugs)
Version: 3.7   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: Luna M1   Edit
Assignee: Thomas Watson CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 412883
  Show dependency tree
 
Reported: 2011-04-15 16:38 EDT by Thomas Watson CLA
Modified: 2013-07-12 14:48 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Watson CLA 2011-04-15 16:38:40 EDT
The code in RegionManager.loadRegionDigraph() currently has the following TODO

// TODO need to validate bundle IDs to make sure they are consistent with current bundles

We must do two things:

1) make sure all the bundle ids in the digraph have associated bundles installed in the framework, if not then remove the bundle ids from their regions.
2) make sure all installed bundles are contained in a region in the digraph, if a bundle is found not be included in a Region then it should be added to the "kernel" region.

We have also discussed the need to check the Equinox State timestamp to see if it has changed since the digraph was persisted.  If the timestamp has not changed then no consistency check is needed.  If the timestamp has changed then we must do a consistency check and we may have to perform a refresh operation to ensure all the bundles in the system are re-resolved with the policy of the digraph.
Comment 1 Thomas Watson CLA 2011-04-15 16:39:40 EDT
Belongs in components.
Comment 2 Thomas Watson CLA 2013-07-12 11:53:07 EDT
I'm not sure why this was never addressed, but it is pretty severe when it surfaces.  We need to fix this in luna and probably backport to Kepler.