Fix ressource leak

[gpl/argeo-slc.git] / runtime / org.argeo.slc.core / src / main / java / org / argeo / slc / core / execution / ProcessThreadGroup.java

diff --git a/runtime/org.argeo.slc.core/src/main/java/org/argeo/slc/core/execution/ProcessThreadGroup.java b/runtime/org.argeo.slc.core/src/main/java/org/argeo/slc/core/execution/ProcessThreadGroup.java
index 6f0143f0fcca651847610471a71ab0945e0d1b74..969c9b93fba5737c75ea723c601402f991b7d7d9 100644 (file)
--- a/runtime/org.argeo.slc.core/src/main/java/org/argeo/slc/core/execution/ProcessThreadGroup.java
+++ b/runtime/org.argeo.slc.core/src/main/java/org/argeo/slc/core/execution/ProcessThreadGroup.java
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2010 Mathieu Baudier <mbaudier@argeo.org>
+ * Copyright (C) 2007-2012 Argeo GmbH
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -13,22 +13,28 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 package org.argeo.slc.core.execution;
 
 import java.util.ArrayList;
 import java.util.List;
+import java.util.concurrent.ArrayBlockingQueue;
+import java.util.concurrent.BlockingQueue;
 
 import org.argeo.slc.execution.ExecutionModulesManager;
 import org.argeo.slc.execution.ExecutionProcess;
 import org.argeo.slc.execution.ExecutionStep;
-import org.argeo.slc.process.SlcExecution;
-import org.argeo.slc.process.SlcExecutionStep;
+import org.springframework.security.Authentication;
+import org.springframework.security.context.SecurityContextHolder;
 
 /** The thread group attached to a given {@link SlcExecution}. */
 public class ProcessThreadGroup extends ThreadGroup {
        private final ExecutionModulesManager executionModulesManager;
        private final ProcessThread processThread;
+       private final Authentication authentication;
+       private final static Integer STEPS_BUFFER_CAPACITY = 5000;
+
+       private BlockingQueue<ExecutionStep> steps = new ArrayBlockingQueue<ExecutionStep>(
+                       STEPS_BUFFER_CAPACITY);
 
        public ProcessThreadGroup(ExecutionModulesManager executionModulesManager,
                        ProcessThread processThread) {
@@ -36,21 +42,22 @@ public class ProcessThreadGroup extends ThreadGroup {
                                + " thread group");
                this.executionModulesManager = executionModulesManager;
                this.processThread = processThread;
+               this.authentication = SecurityContextHolder.getContext()
+                               .getAuthentication();
        }
 
-       // public SlcExecution getSlcProcess() {
-       // return processThread.getSlcProcess();
-       // }
+       public Authentication getAuthentication() {
+               return authentication;
+       }
 
        public void dispatchAddStep(ExecutionStep step) {
-               // legacy
                ExecutionProcess slcProcess = processThread.getProcess();
-               if (slcProcess instanceof SlcExecution)
-                       ((SlcExecution) slcProcess).getSteps().add((SlcExecutionStep) step);
-
                List<ExecutionStep> steps = new ArrayList<ExecutionStep>();
                steps.add(step);
-               dispatchAddSteps(steps);
+               // TODO clarify why we don't dispatch steps, must be a reason
+               // dispatchAddSteps(steps);
+               slcProcess.addSteps(steps);
+               this.steps.add(step);
        }
 
        public void dispatchAddSteps(List<ExecutionStep> steps) {
@@ -58,4 +65,8 @@ public class ProcessThreadGroup extends ThreadGroup {
                executionModulesManager.dispatchAddSteps(slcProcess, steps);
        }
 
+       public BlockingQueue<ExecutionStep> getSteps() {
+               return steps;
+       }
+
 }