Noting JENKINS-37154.

jenkinsci · Aug 3, 2016 · 6efdc1f · 6efdc1f
1 parent 973cd1e
commit 6efdc1f
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/src/main/java/org/jenkinsci/plugins/workflow/support/steps/input/InputAction.java b/src/main/java/org/jenkinsci/plugins/workflow/support/steps/input/InputAction.java
@@ -62,11 +62,7 @@ private synchronized void loadExecutions() {
                 }
             }
             if (execution != null) {
-                // Futures.addCallback is the safer way to iterate, but we need to know that the result is in.
-                // And we cannot start the calculation during onLoad because we are still inside WorkflowRun.onLoad and the CpsFlowExecution is not yet initialized.
-                // WorkflowRun has getExecutionPromise but that is not accessible via API.
-                // Even if we add such an API to FlowExecutionOwner, and look for FlowExecutionOwner.Executable,
-                // FlowExecutionList would need an API for getting owners without blocking on the execution.
+                // TODO JENKINS-37154 we would rather not block here
                 for (StepExecution se : execution.getCurrentExecutions(true).get()) {
                     if (se instanceof InputStepExecution) {
                         InputStepExecution ise = (InputStepExecution) se;