From 5108908aa6d2169af5a15b156791e53afeeb08be Mon Sep 17 00:00:00 2001 From: David Dykstal Date: Thu, 20 Dec 2012 18:04:01 -0600 Subject: [PATCH] Test moving the scheduling of the RSEInitJob to the RSEUIPlugin The RSEInitJob loads profiles which instantiates subsystem configurations. If these subsystem configurations are located in plugins that depend on UI plugins then we run the risk of activating these plugins before the platform is ready resulting in load errors of various kinds. There appears to be a change in job scheduling timing that makes this error more apparent. --- .../schema/modelInitializers.exsd | 30 +++++++++---------- .../org/eclipse/rse/core/RSECorePlugin.java | 2 ++ .../eclipse/rse/internal/core/RSEInitJob.java | 5 ++-- .../UI/org/eclipse/rse/ui/RSEUIPlugin.java | 3 ++ 4 files changed, 23 insertions(+), 17 deletions(-) diff --git a/rse/plugins/org.eclipse.rse.core/schema/modelInitializers.exsd b/rse/plugins/org.eclipse.rse.core/schema/modelInitializers.exsd index 1027cb2bd70..d56951971de 100644 --- a/rse/plugins/org.eclipse.rse.core/schema/modelInitializers.exsd +++ b/rse/plugins/org.eclipse.rse.core/schema/modelInitializers.exsd @@ -2,19 +2,19 @@ - + - + - A model initializer is used to create supplemental connections and filter pools after the RSE model has been restored from its persistent form. The base RSE uses this extension point to supply the "Local" connection if one has not already been created for this workspace. Other extenders of RSE may use this to augment the mode in other ways. An initializer is run at the end of the initialization job begun by RSE when it is activated. + A model initializer is used to create supplemental connections and filter pools after the RSE model has been restored from its persistent form. The base RSE uses this extension point to supply the "Local" connection if one has not already been created for this workspace. Other extenders of RSE may use this to augment the model in other ways. An initializer is run at the end of the initialization job begun by RSE when it is activated. - + - + @@ -39,9 +39,9 @@ - + - + @@ -54,9 +54,9 @@ A class that extends <b>org.eclipse.rse.core.IRSEModelInitializer</b>. - + - + @@ -78,18 +78,18 @@ - + - + RSE 3.0 - + - + An example taken from the plug-in org.eclipse.rse.ui: <pre> @@ -103,9 +103,9 @@ - + - + Copyright (c) 2008 IBM Corporation and others. All rights reserved. This program and the accompanying materials are made available under the terms diff --git a/rse/plugins/org.eclipse.rse.core/src/org/eclipse/rse/core/RSECorePlugin.java b/rse/plugins/org.eclipse.rse.core/src/org/eclipse/rse/core/RSECorePlugin.java index 3ea90e23e66..9ce9edf3f66 100644 --- a/rse/plugins/org.eclipse.rse.core/src/org/eclipse/rse/core/RSECorePlugin.java +++ b/rse/plugins/org.eclipse.rse.core/src/org/eclipse/rse/core/RSECorePlugin.java @@ -307,8 +307,10 @@ public class RSECorePlugin extends Plugin { public void start(BundleContext context) throws Exception { super.start(context); registerKeystoreProviders(); + /* RSEInitJob job = RSEInitJob.getInstance(); job.schedule(); + */ } /* diff --git a/rse/plugins/org.eclipse.rse.core/src/org/eclipse/rse/internal/core/RSEInitJob.java b/rse/plugins/org.eclipse.rse.core/src/org/eclipse/rse/internal/core/RSEInitJob.java index 551e1960159..0b0cbb6c400 100644 --- a/rse/plugins/org.eclipse.rse.core/src/org/eclipse/rse/internal/core/RSEInitJob.java +++ b/rse/plugins/org.eclipse.rse.core/src/org/eclipse/rse/internal/core/RSEInitJob.java @@ -11,6 +11,7 @@ * David Dykstal (IBM) = [226958] add status values to waitForInitCompletion(phase) * David Dykstal (IBM) - [235581] Initialize RSE should be a daemon job * David McKnight (IBM) - [283033] remoteFileTypes extension point should include "xml" type + * David Dykstal (IBM) - [] RSEInitJob runs too early ********************************************************************************/ package org.eclipse.rse.internal.core; @@ -230,10 +231,10 @@ public final class RSEInitJob extends Job { return ResourcesPlugin.getEncoding(); } }); -*/ + */ initializerPhase.done(result); - // finish up - propogate cancel if necessary + // finish up - propagate cancel if necessary if (monitor.isCanceled()) { result = Status.CANCEL_STATUS; } else { diff --git a/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/RSEUIPlugin.java b/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/RSEUIPlugin.java index 98daf13a0cf..52a8ba2c862 100644 --- a/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/RSEUIPlugin.java +++ b/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/RSEUIPlugin.java @@ -53,6 +53,7 @@ import org.eclipse.rse.core.model.ISystemProfileManager; import org.eclipse.rse.core.model.ISystemRegistry; import org.eclipse.rse.core.subsystems.ISubSystemConfiguration; import org.eclipse.rse.core.subsystems.ISubSystemConfigurationProxy; +import org.eclipse.rse.internal.core.RSEInitJob; import org.eclipse.rse.internal.core.model.SystemProfileManager; import org.eclipse.rse.internal.core.model.SystemRegistry; import org.eclipse.rse.internal.ui.RSESystemTypeAdapterFactory; @@ -385,6 +386,8 @@ public class RSEUIPlugin extends SystemBasePlugin public void start(BundleContext context) throws Exception { super.start(context); + + RSEInitJob.getInstance().schedule(); messageFile = getMessageFile("systemmessages.xml"); //$NON-NLS-1$ defaultMessageFile = getDefaultMessageFile("systemmessages.xml"); //$NON-NLS-1$