code cleanup and logging
diff --git a/plugins/org.eclipse.amalgam.discovery.modeling/src/org/eclipse/amalgam/discovery/modeling/handlers/ModelingPackageDiscovery.java b/plugins/org.eclipse.amalgam.discovery.modeling/src/org/eclipse/amalgam/discovery/modeling/handlers/ModelingPackageDiscovery.java
index 03bfc20..05e06c3 100644
--- a/plugins/org.eclipse.amalgam.discovery.modeling/src/org/eclipse/amalgam/discovery/modeling/handlers/ModelingPackageDiscovery.java
+++ b/plugins/org.eclipse.amalgam.discovery.modeling/src/org/eclipse/amalgam/discovery/modeling/handlers/ModelingPackageDiscovery.java
@@ -20,7 +20,9 @@
 import org.eclipse.amalgam.discovery.ui.viewer.DiscoveryContentProvider;
 import org.eclipse.amalgam.discovery.ui.wizards.DiscoveryWizard;
 import org.eclipse.core.runtime.IProgressMonitor;
+import org.eclipse.core.runtime.IStatus;
 import org.eclipse.core.runtime.OperationCanceledException;
+import org.eclipse.core.runtime.Status;
 import org.eclipse.emf.common.util.URI;
 import org.eclipse.jface.dialogs.MessageDialog;
 import org.eclipse.jface.operation.IRunnableWithProgress;
@@ -101,8 +103,11 @@
 			}
 		} catch (InvocationTargetException e1) {
 			if (!(e1.getCause() instanceof OperationCanceledException)) {
-				// TODO Auto-generated catch block
-				e1.printStackTrace();
+				Activator
+						.getDefault()
+						.getLog()
+						.log(new Status(IStatus.ERROR, Activator.PLUGIN_ID, e1
+								.getMessage(), e1));
 			}
 		} catch (InterruptedException e1) {
 			/*
diff --git a/plugins/org.eclipse.amalgam.discovery.ui/src/org/eclipse/amalgam/discovery/ui/viewer/DiscoveryContentProvider.java b/plugins/org.eclipse.amalgam.discovery.ui/src/org/eclipse/amalgam/discovery/ui/viewer/DiscoveryContentProvider.java
index 3f36a41..cf93643 100644
--- a/plugins/org.eclipse.amalgam.discovery.ui/src/org/eclipse/amalgam/discovery/ui/viewer/DiscoveryContentProvider.java
+++ b/plugins/org.eclipse.amalgam.discovery.ui/src/org/eclipse/amalgam/discovery/ui/viewer/DiscoveryContentProvider.java
@@ -13,7 +13,6 @@
 import org.eclipse.amalgam.discovery.DiscoveryDefinition;
 import org.eclipse.amalgam.discovery.core.InstallationStatusUpdater;
 import org.eclipse.core.runtime.IProgressMonitor;
-import org.eclipse.emf.common.util.Monitor;
 
 /**
  *