Skip to content

Commit 2128282

Browse files
author
zhangyongxiang.alpha
committed
fix executeEngineManager
1 parent 167f8f3 commit 2128282

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

amoro-ams/src/main/java/org/apache/amoro/server/AmoroServiceContainer.java

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@
3434
import org.apache.amoro.config.shade.utils.ConfigShadeUtils;
3535
import org.apache.amoro.exception.AmoroRuntimeException;
3636
import org.apache.amoro.process.ActionCoordinator;
37+
import org.apache.amoro.process.ExecuteEngine;
3738
import org.apache.amoro.process.ProcessFactory;
3839
import org.apache.amoro.server.catalog.CatalogManager;
3940
import org.apache.amoro.server.catalog.DefaultCatalogManager;
@@ -241,14 +242,16 @@ public void startOptimizingService() throws Exception {
241242
TableProcessFactoryManager tableProcessFactoryManager = new TableProcessFactoryManager();
242243
tableProcessFactoryManager.initialize();
243244
List<ProcessFactory> processFactories = tableProcessFactoryManager.installedPlugins();
245+
ExecuteEngineManager executeEngineManager = new ExecuteEngineManager();
246+
executeEngineManager.initialize();
247+
List<ExecuteEngine> executeEngines = executeEngineManager.installedPlugins();
248+
processFactories.forEach(
249+
c -> c.availableExecuteEngines(executeEngines));
244250

245251
DefaultTableRuntimeFactory defaultRuntimeFactory = new DefaultTableRuntimeFactory();
246252
defaultRuntimeFactory.initialize(processFactories);
247253

248254
List<ActionCoordinator> actionCoordinators = defaultRuntimeFactory.supportedCoordinators();
249-
ExecuteEngineManager executeEngineManager = new ExecuteEngineManager();
250-
processFactories.forEach(
251-
c -> c.availableExecuteEngines(executeEngineManager.installedPlugins()));
252255

253256
tableService = new DefaultTableService(serviceConfig, catalogManager, defaultRuntimeFactory);
254257
processService = new ProcessService(tableService, actionCoordinators, executeEngineManager);

amoro-ams/src/main/java/org/apache/amoro/server/process/ProcessService.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,6 @@ private void initialize(List<TableRuntime> tableRuntimes) {
128128
actionCoordinator.action().getName(),
129129
new ActionCoordinatorScheduler(actionCoordinator, tableService, ProcessService.this));
130130
}
131-
executeEngineManager.initialize();
132131
executeEngineManager
133132
.installedPlugins()
134133
.forEach(

0 commit comments

Comments
 (0)