Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 335920

Summary:	[remoteserviceadmin] fix EndpointDescriptionLocator registering two service listeners
Product:	[RT] ECF	Reporter:	Scott Lewis <slewis>
Component:	ecf.remoteservices	Assignee:	Scott Lewis <slewis>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	bugs.eclipse.org
Version:	3.4.0
Target Milestone:	3.5.0
Hardware:	PC
OS:	Windows XP
Whiteboard:

Description Scott Lewis

2011-01-31 23:36:05 EST

Currently, the RSA impl registers two EndpointDescriptionLocator instances upon start/configuration.  This is incorrect...it should only register one for each IDiscoveryLocator discovery provider.

Comment 1 Scott Lewis

2011-01-31 23:38:28 EST

assigning.

Comment 2 Scott Lewis

2011-01-31 23:43:03 EST

Fix released to master.  Resolving.

Comment 3 Markus Kuppe

2011-02-01 03:12:24 EST

Hi Scott,

without having looked at the code I wonder why you need on EndpointDescriptionLocator per IDiscoveryLocator. With enhancement bug #328011 can't you just register one IServiceListener|IServiceTypeListener with the OSGi registry and be notified automatically?

Comment 4 Scott Lewis

2011-02-01 10:31:16 EST

(In reply to comment #3)
> Hi Scott,
> 
> without having looked at the code I wonder why you need on
> EndpointDescriptionLocator per IDiscoveryLocator. 
>With enhancement bug #328011
> can't you just register one IServiceListener|IServiceTypeListener with the OSGi
> registry and be notified automatically?

Yes, this change/simplification could be made (given 328011).  I haven't had the time/capacity to make it.  If you want to open an enhancement to that effect, we'll try to get to it when possible.

Comment 5 Markus Kuppe

2011-02-01 10:46:52 EST

Done, bug #335977