Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 342889 - [region] add API to copy and replace a digraph
Summary: [region] add API to copy and replace a digraph
Status: RESOLVED FIXED
Alias: None
Product: Equinox
Classification: Eclipse Project
Component: Components (show other bugs)
Version: 3.7   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Thomas Watson CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-14 16:29 EDT by Thomas Watson CLA
Modified: 2011-04-15 09:49 EDT (History)
1 user (show)

See Also:

Attachments
patch with API, impl and tests (3.14 KB, patch)
2011-04-15 09:46 EDT, Thomas Watson CLA 		no flags Details | Diff
patch with API, impl and tests (really this time) (8.43 KB, patch)
2011-04-15 09:47 EDT, Thomas Watson 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 Thomas Watson CLA 2011-04-14 16:29:11 EDT 
For scenarios that need to update an existing digraph's set of connections the current API requires a region to be removed and then added back followed by reconnecting all the necessary regions with the new connection policy.

The problem with this approach is there is no atomic way to do this.  This can cause windows of time where a bundle does not belong to any region while we are changing the region's connections.  It would be good if we had a way to obtain a copy of the digraph associated with the framework.  Then we could manipulate the copy of the digraph off line.  Finally we would need a way to replace the "live" digraph with the content of the copy.
Comment 1 Thomas Watson CLA 2011-04-15 09:46:07 EDT 
Created attachment 193373 [details]
patch with API, impl and tests
Comment 2 Thomas Watson CLA 2011-04-15 09:47:28 EDT 
Created attachment 193374 [details]
patch with API, impl and tests (really this time)

Opps, here is the correct patch.
Comment 3 Thomas Watson CLA 2011-04-15 09:49:00 EDT 
Patch released.