mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-04-29 19:45:01 +02:00
Code Issues Wiki Activity

[305376] - [concurrent] RejectedExecutionExeption is not properly handled in RequestMonitor

Browse source
This commit is contained in:
Pawel Piech 2010-03-10 19:11:17 +00:00
parent c389d6d748
commit afa7312595
1 changed files with 12 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
dsf/org.eclipse.cdt.dsf/src/org/eclipse/cdt/dsf/concurrent/RequestMonitor.java
View file

@ -451,15 +451,23 @@ public class RequestMonitor extends DsfExecutable {
* Default handler for when the executor supplied in the constructor
* rejects the runnable that is submitted invoke this request monitor.
* This usually happens only when the executor is shutting down.
* <p>
* The default handler creates a new error status for the rejected
* execution and propagates it to the client or logs it.
*/
protected void handleRejectedExecutionException() {
MultiStatus logStatus = new MultiStatus(DsfPlugin.PLUGIN_ID, IDsfStatusConstants.INTERNAL_ERROR, "Request for monitor: '" + toString() + "' resulted in a rejected execution exception.", null); //$NON-NLS-1$ //$NON-NLS-2$
logStatus.merge(getStatus());
IStatus rejectedStatus = new Status(IStatus.ERROR, DsfPlugin.PLUGIN_ID, IDsfStatusConstants.INTERNAL_ERROR, "Request for monitor: '" + toString() + "' resulted in a rejected execution exception.", null); //$NON-NLS-1$ //$NON-NLS-2$
if (!getStatus().isOK()) {
DsfMultiStatus multiStatus = new DsfMultiStatus(DsfPlugin.PLUGIN_ID, 0, "Composite status", null); //$NON-NLS-1$
multiStatus.merge(getStatus());
multiStatus.merge(rejectedStatus);
rejectedStatus = multiStatus;
}
if (fParentRequestMonitor != null) {
fParentRequestMonitor.setStatus(logStatus);
fParentRequestMonitor.setStatus(rejectedStatus);
fParentRequestMonitor.done();
} else {
DsfPlugin.getDefault().getLog().log(logStatus);
DsfPlugin.getDefault().getLog().log(rejectedStatus);
}
}