Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 327060 - [discovery][zookeeper] location URI should include port
Summary: [discovery][zookeeper] location URI should include port
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.4.0   Edit
Assignee: Markus Kuppe CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 327034
Blocks:
  Show dependency tree
 
Reported: 2010-10-06 01:10 EDT by Scott Lewis CLA
Modified: 2010-10-20 05:16 EDT (History)
2 users (show)

See Also:

Attachments
ServiceInfo Location uri fix (1.14 KB, patch)
2010-10-19 11:30 EDT, Ahmed Aadel CLA 		no flags Details | Diff
mylyn/context/zip (904 bytes, application/octet-stream)
2010-10-20 05:16 EDT, Markus Kuppe CLA 		no flags Details
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 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