Bug 230180 - [remotesvcs] Provide org.eclipse.ecf.discovery.IDiscoveryContainerAdapter#unregisterAllServices()
Summary: [remotesvcs] Provide org.eclipse.ecf.discovery.IDiscoveryContainerAdapter#unr...
Status: RESOLVED FIXED
Alias: None
Product: ECF
Classification: RT
Component: ecf.discovery (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 3.0.0   Edit
Assignee: Markus Kuppe CLA
QA Contact:
URL:
Whiteboard:
Keywords: plan
Depends on: 257861
Blocks: 251330
  Show dependency tree
 
Reported: 2008-05-05 07:31 EDT by Markus Kuppe CLA
Modified: 2009-02-25 10:02 EST (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 Markus Kuppe CLA 2008-05-05 07:31:04 EDT 
org.eclipse.ecf.discovery.IDiscoveryContainerAdapter may provide API to unregister all currently registered IServiceInfos at once.
Comment 1 Scott Lewis CLA 2008-10-23 15:02:46 EDT 
Assigning to Markus for target milestone 2.1.0.
Comment 2 Markus Kuppe CLA 2008-10-23 17:18:45 EDT 
Resetting to 3.0 since this is a breaking API change.
Comment 3 Scott Lewis CLA 2008-12-15 14:37:05 EST 
I'm +1 on this API addition.
Comment 4 Markus Kuppe CLA 2008-12-16 06:00:14 EST 
Problem is, that this functionality cannot be offered by all providers. E.g. jSLP 1.x is stateless. So the question comes up how to handle it:

a) throw a checked org.eclipse.ecf.core.util.ECFException (relates to bug #257864)
c) throw an unchecked java.lang.UnsupportedOperationException
b) make it a no op for jSLP
Comment 5 Markus Kuppe CLA 2009-02-25 09:23:23 EST 
Method has been added to IDiscoveryLocator with the impl in AbstractDiscoveryContainerAdapter throwing java.lang.UnsupportedOperationException.
Comment 6 Markus Kuppe CLA 2009-02-25 09:28:16 EST 
org.eclipse.ecf.provider.discovery.CompositeDiscoveryContainer implementation simply delegates to nested containers.
Comment 7 Markus Kuppe CLA 2009-02-25 10:02:02 EST 
Released to HEAD