Bug 226074 - [dstore] Add status return codes for getStatus() API in ConnectionEstablisher
Summary: [dstore] Add status return codes for getStatus() API in ConnectionEstablisher
Status: RESOLVED FIXED
Alias: None
Product: Target Management
Classification: Tools
Component: RSE (show other bugs)
Version: 3.0   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 3.0 M6   Edit
Assignee: David McKnight CLA
QA Contact: Martin Oberhuber CLA
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2008-04-08 02:29 EDT by Noriaki Takatsu CLA
Modified: 2011-05-25 07:28 EDT (History)
2 users (show)

See Also:

Attachments
Patch for the process of getStatus() API. (2.02 KB, patch)
2008-04-08 02:38 EDT, Noriaki Takatsu CLA 		mober.at+eclipse: iplog+
Details | Diff
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 Noriaki Takatsu CLA 2008-04-08 02:29:20 EDT 
In the current release of IBM RSE server (invoked as a daemon), 
System.err is used for ConnectionEstablisher to inform the daemon of an error 
that occurred in the ConnectionEstablisher.
This System.err mechanism does not work when the Server class runs as a thread.
So, I added an API named getStatus() in 220126 but the needed process for getStatus() was not included in the 220126.
This patch is the needed process for getStatus() API.
Comment 1 Noriaki Takatsu CLA 2008-04-08 02:38:01 EDT 
Created attachment 95151 [details]
Patch for the process of getStatus() API.
Comment 2 Martin Oberhuber CLA 2008-04-08 04:13:20 EDT 
Sounds like an API addition. Mind you, today is the really last day I'm accepting API changes.
Comment 3 Noriaki Takatsu CLA 2008-04-08 04:34:11 EDT 
getStatus() API was already added in 220126, but the needed process was not included in 220126 patch.
So this 226074 is not API addition but enhancement of the existing API.
Comment 4 Martin Oberhuber CLA 2008-04-08 04:54:22 EDT 
Ah right, I misunderstood. Changing summary, old value was:
Add process for getStatus() API in ConnectionEstablisher

Dave the fix seems simple enought for M6.
Comment 5 David McKnight CLA 2008-04-08 10:51:42 EDT 
I've committed the change to cvs.
Comment 6 Noriaki Takatsu CLA 2008-04-23 05:44:05 EDT 
Legal Message:
 I, {Noriaki Takatsu}, declare that I developed attached code from scratch, 
 without referencing any 3rd party materials except material licensed under 
 the EPL. 
 I am authorized by my employer to make this contribution under the EPL.