Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 330906 - RepositorySynchronizer Thread might die
Summary: RepositorySynchronizer Thread might die
Status: CLOSED FIXED
Alias: None
Product: EMF
Classification: Modeling
Component: cdo.core (show other bugs)
Version: 4.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Pascal Lehmann CLA
QA Contact: Eike Stepper CLA
URL:
Whiteboard: offline-01
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-23 07:06 EST by Pascal Lehmann CLA
Modified: 2011-06-23 03:39 EDT (History)
0 users

See Also:

Attachments
Patch v1 - ready to be committed (1.23 KB, patch)
2010-11-29 04:53 EST, Eike Stepper CLA 		no flags 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 Pascal Lehmann CLA 2010-11-23 07:06:38 EST 
The ReplicateRunnable lacks a try-catch block, so whenever an Exception is thrown (eg. problem caused by session disconnects) the QueueRunner thread dies and no more QueueRunnables will be processed.

Maybe the QueueRunner should protect itself?
Comment 1 Eike Stepper CLA 2010-11-29 04:53:33 EST 
Created attachment 184018 [details]
Patch v1 - ready to be committed

Fixed by not letting the Worker break its run() loop on exceptions from a work item.
Comment 2 Pascal Lehmann CLA 2010-11-29 06:43:12 EST 
Committed Patch v1 to HEAD.
Comment 3 Eike Stepper CLA 2011-06-23 03:39:06 EDT 
Available in R20110608-1407