Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 341578 - [dstore] ServerLogger is looped when IOError happens
Summary: [dstore] ServerLogger is looped when IOError happens
Status: RESOLVED FIXED
Alias: None
Product: Target Management
Classification: Tools
Component: RSE (show other bugs)
Version: 3.2.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.3 M7   Edit
Assignee: David McKnight CLA
QA Contact: Martin Oberhuber CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 341613
  Show dependency tree
 
Reported: 2011-04-01 03:39 EDT by Noriaki Takatsu CLA
Modified: 2011-05-18 22:55 EDT (History)
2 users (show)

See Also:

Attachments
Fix a loop when IOError happens in ServerLogger (789 bytes, patch)
2011-04-01 03:43 EDT, Noriaki Takatsu CLA 		mober.at+eclipse: iplog+
dmcknigh: review+
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 2011-04-01 03:39:06 EDT 
When IO error happens in creating a log file,  a loop occurs in getLogFile(String preferredName) method  of SerevrLogger class.
Comment 1 Noriaki Takatsu CLA 2011-04-01 03:43:15 EDT 
Created attachment 192336 [details]
Fix a loop when IOError happens in ServerLogger
Comment 2 Onno Van den Troost CLA 2011-04-01 07:50:23 EDT 
please backport to 321
Comment 3 David McKnight CLA 2011-04-01 08:52:30 EDT 
Comment on attachment 192336 [details]
Fix a loop when IOError happens in ServerLogger

Could you please add a legal disclaimer for the patch?
Comment 4 Noriaki Takatsu CLA 2011-04-01 23:33:05 EDT 
Legal Message:
 I, {Noriaki Takatsu}, declare that I developed the 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.
Comment 5 David McKnight CLA 2011-04-04 09:05:09 EDT 
Thanks for providing a disclaimer.  I've committed the change (with the copyright line) to cvs.