Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 335920 - [remoteserviceadmin] fix EndpointDescriptionLocator registering two service listeners
Summary: [remoteserviceadmin] fix EndpointDescriptionLocator registering two service l...
Status: RESOLVED FIXED
Alias: None
Product: ECF
Classification: RT
Component: ecf.remoteservices (show other bugs)
Version: 3.4.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.5.0   Edit
Assignee: Scott Lewis CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-31 23:36 EST by Scott Lewis CLA
Modified: 2011-02-01 10:46 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 Scott Lewis CLA 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 CLA 2011-01-31 23:38:28 EST 
assigning.
Comment 2 Scott Lewis CLA 2011-01-31 23:43:03 EST 
Fix released to master.  Resolving.
Comment 3 Markus Kuppe CLA 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 CLA 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 CLA 2011-02-01 10:46:52 EST 
Done, bug #335977