From 405a756d6a6cd9e613d10bc7f2e382f666c01df5 Mon Sep 17 00:00:00 2001
From: John Ross
Date: Tue, 5 Mar 2013 14:35:23 -0600
Subject: Bug 402478 - [coordinator] Be less aggressive with log messages when
 processing orphaned coordinations.

Changed unexpected exception logging when failing an orphaned coordination from ERROR to WARNING.

Changed unexpected exception logging when ending an orphaned coordination from ERROR to WARNING.

Changed unexpected coordination exception failure cause logging from ERROR to DEBUG.

Exception instead of Throwable now caught.

Updated copyrights, including one from previous change in CoordinationImpl.
---
 .../equinox/coordinator/CoordinationWeakReference.java       | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/bundles/org.eclipse.equinox.coordinator/src/org/eclipse/equinox/coordinator/CoordinationWeakReference.java b/bundles/org.eclipse.equinox.coordinator/src/org/eclipse/equinox/coordinator/CoordinationWeakReference.java
index 7c7d90ca4..33d5ef4ae 100644
--- a/bundles/org.eclipse.equinox.coordinator/src/org/eclipse/equinox/coordinator/CoordinationWeakReference.java
+++ b/bundles/org.eclipse.equinox.coordinator/src/org/eclipse/equinox/coordinator/CoordinationWeakReference.java
@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2011 IBM Corporation and others.
+ * Copyright (c) 2011, 2013 IBM Corporation and others.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
@@ -28,8 +28,8 @@ public class CoordinationWeakReference extends WeakReference<CoordinationReferen
 			try {
 				c.fail(Coordination.ORPHANED);
 			}
-			catch (Throwable t) {
-				c.getLogService().log(LogService.LOG_ERROR, NLS.bind(Messages.OrphanedCoordinationError, c.getName(), c.getId()), t);
+			catch (Exception e) {
+				c.getLogService().log(LogService.LOG_WARNING, NLS.bind(Messages.OrphanedCoordinationError, c.getName(), c.getId()), e);
 			}
 			finally {
 				try {
@@ -39,10 +39,10 @@ public class CoordinationWeakReference extends WeakReference<CoordinationReferen
 					// This is expected since we already failed the coordination...
 					if (!Coordination.ORPHANED.equals(e.getCause()))
 						// ...but only if the cause is ORPHANED.
-						c.getLogService().log(LogService.LOG_ERROR, NLS.bind(Messages.OrphanedCoordinationError, c.getName(), c.getId()), e);
+						c.getLogService().log(LogService.LOG_DEBUG, NLS.bind(Messages.OrphanedCoordinationError, c.getName(), c.getId()), e);
 				}
-				catch (Throwable t) {
-					c.getLogService().log(LogService.LOG_ERROR, NLS.bind(Messages.OrphanedCoordinationError, c.getName(), c.getId()), t);
+				catch (Exception e) {
+					c.getLogService().log(LogService.LOG_WARNING, NLS.bind(Messages.OrphanedCoordinationError, c.getName(), c.getId()), e);
 				}
 			}
 		}
-- 
cgit v1.2.3