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

Bug 327060

Summary: [discovery][zookeeper] location URI should include port
Product: [RT] ECF Reporter: Scott Lewis <slewis>
Component: ecf.remoteservicesAssignee: Markus Kuppe <bugs.eclipse.org>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: ahmed.aadel, bhunt
Version: 3.4.0   
Target Milestone: 3.4.0   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Bug Depends on: 327034    
Bug Blocks:    
Attachments:
Description Flags
ServiceInfo Location uri fix
none
mylyn/context/zip none

Description Scott Lewis CLA 2010-10-06 01:10:18 EDT 
As described in ecf-dev mailing list posting

http://dev.eclipse.org/mhonarc/lists/ecf-dev/msg04290.html

the zookeeper location URI should include the port of interest for a given service.

Note as described in bug 327034 comment 3 this bug provides part of the solution to the problem described on the mailing list.
Comment 1 Ahmed Aadel CLA 2010-10-19 11:30:24 EDT 
Created attachment 181193 [details]
ServiceInfo Location uri fix

Hi Scott/Markus

Here is the patch. Can you please git it into the repos after you approve? It is only a 5 character change.
Comment 2 Markus Kuppe CLA 2010-10-20 05:16:06 EDT 
Patch looks good to me. Bryan, please give it a spin too?
Fix released to master.
Appears in build #54 [0].

[0] https://build.ecf-project.org/hudson/job/C-HEAD-discovery.zookeeper.feature/54
Comment 3 Markus Kuppe CLA 2010-10-20 05:16:08 EDT 
Created attachment 181266 [details]
mylyn/context/zip