TMF: bug 440998: Set active task to null in TmfRequestExecutor when completed

This avoids keeping references to the request, which itself may contain
references to rather big objects.

Change-Id: Ib82fbea5cdc6b73a0a76d51bc2e960d0d223927a
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/30903
Tested-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

diff --git a/org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/internal/tmf/core/request/TmfRequestExecutor.java b/org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/internal/tmf/core/request/TmfRequestExecutor.java
index a308486d37c82a6aa607fe9f1c2fa3d941fbd8b6..96e1c90cd524aaa90939ac48e691797961df301a 100644 (file)
--- a/org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/internal/tmf/core/request/TmfRequestExecutor.java
+++ b/org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/internal/tmf/core/request/TmfRequestExecutor.java
@@ -172,6 +172,7 @@ public class TmfRequestExecutor implements Executor {
                 schedule();
             } else if (fActiveTask.getExecType() == ExecutionType.FOREGROUND) {
                 if (fActiveTask.getThread().isCompleted()) {
+                    fActiveTask = null;
                     schedule();
                 } else {
                     if (hasTasks()) {
@@ -183,6 +184,7 @@ public class TmfRequestExecutor implements Executor {
 
             } else if (fActiveTask.getExecType() == ExecutionType.BACKGROUND) {
                 if (fActiveTask.getThread().isCompleted()) {
+                    fActiveTask = null;
                     schedule();
                 } else {
                     if (hasTasks()) {