diff options
author | 2017-11-21 16:37:13 -0800 | |
---|---|---|
committer | 2017-11-21 16:38:50 -0800 | |
commit | 7967f3387e63723bb36b974f83a816265d404c38 (patch) | |
tree | 1533cc8bd2d63e1db9426d8795df902d104a3d1a /src/main/java/com/google/devtools/build/lib/rules/test | |
parent | a2a3d8ca6634abb8f8dc184edb4ea01543de8505 (diff) |
Return SpawnResults using a List instead of a Set.
Currently we don't care about the list order of SpawnResults. However, we may care about the list order later. Also, if the equals() method for SpawnResults is ever changed then it may be problematic to return SpawnResults in a Set.
Aside: ActionResults use SpawnResults to calculate cumulative execution times for Actions, and may provide other metrics in future.
RELNOTES: None.
PiperOrigin-RevId: 176579460
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/test')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/test/ExclusiveTestStrategy.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/test/ExclusiveTestStrategy.java b/src/main/java/com/google/devtools/build/lib/rules/test/ExclusiveTestStrategy.java index dd127ca8ec..66e25763ec 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/test/ExclusiveTestStrategy.java +++ b/src/main/java/com/google/devtools/build/lib/rules/test/ExclusiveTestStrategy.java @@ -23,7 +23,7 @@ import com.google.devtools.build.lib.analysis.test.TestRunnerAction; import com.google.devtools.build.lib.vfs.Path; import com.google.devtools.build.lib.view.test.TestStatus.TestResultData; import java.io.IOException; -import java.util.Set; +import java.util.List; /** * Test strategy wrapper called 'exclusive'. It should delegate to a test strategy for local @@ -41,7 +41,7 @@ public class ExclusiveTestStrategy implements TestActionContext { } @Override - public Set<SpawnResult> exec( + public List<SpawnResult> exec( TestRunnerAction action, ActionExecutionContext actionExecutionContext) throws ExecException, InterruptedException { return parent.exec(action, actionExecutionContext); |