diff options
author | 2018-07-24 05:19:34 -0700 | |
---|---|---|
committer | 2018-07-24 05:21:00 -0700 | |
commit | 848375823f64bdeeb00ba5da0c4ded08fded1ddf (patch) | |
tree | 349730231346ca42c26fe66ed2d42cc6e5dceae1 /src/main/java/com/google/devtools/build/lib/skyframe/TargetPatternPhaseValue.java | |
parent | 0b2f868885b1a7162d85ce43d28383a759ec4e92 (diff) |
Move LoadingPhaseCompleteEvent posting to TargetPatternPhaseFunction
Also simplify LoadingPhaseCompleteEvent, and SkyframeExecutor, and remove
LoadingCallback, which is unnecessary now that we only have a single
implementation (previously LoadingPhaseRunner).
This also removes some of the excessive Skyframe calls introduced by
https://github.com/bazelbuild/bazel/commit/1067310e18cb9ac203110726de0be53bdc403cea, and prepares for interleaving target pattern eval and loading.
PiperOrigin-RevId: 205813197
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/skyframe/TargetPatternPhaseValue.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/skyframe/TargetPatternPhaseValue.java | 30 |
1 files changed, 0 insertions, 30 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/TargetPatternPhaseValue.java b/src/main/java/com/google/devtools/build/lib/skyframe/TargetPatternPhaseValue.java index ea7b37a16a..7670dc0588 100644 --- a/src/main/java/com/google/devtools/build/lib/skyframe/TargetPatternPhaseValue.java +++ b/src/main/java/com/google/devtools/build/lib/skyframe/TargetPatternPhaseValue.java @@ -50,10 +50,6 @@ public final class TargetPatternPhaseValue implements SkyValue { @Nullable private final ImmutableSet<Label> testsToRunLabels; private final boolean hasError; private final boolean hasPostExpansionError; - - // This field is only for the purposes of generating the LoadingPhaseCompleteEvent. - // TODO(ulfjack): Support EventBus event posting in Skyframe, and remove this code again. - private final ImmutableSet<Label> removedTargetLabels; private final String workspaceName; TargetPatternPhaseValue( @@ -61,13 +57,11 @@ public final class TargetPatternPhaseValue implements SkyValue { ImmutableSet<Label> testsToRunLabels, boolean hasError, boolean hasPostExpansionError, - ImmutableSet<Label> removedTargetLabels, String workspaceName) { this.targetLabels = targetLabels; this.testsToRunLabels = testsToRunLabels; this.hasError = hasError; this.hasPostExpansionError = hasPostExpansionError; - this.removedTargetLabels = removedTargetLabels; this.workspaceName = workspaceName; } @@ -105,28 +99,6 @@ public final class TargetPatternPhaseValue implements SkyValue { return hasPostExpansionError; } - /** - * Returns a set of targets that were present on the command line but got expanded during the - * loading phase (currently these are only test suites; this set is always empty when <code> - * --expand_test_suites=false</code>. - */ - public ImmutableSet<Target> getRemovedTargets( - ExtendedEventHandler eventHandler, PackageManager packageManager) { - return removedTargetLabels - .stream() - .map( - (label) -> { - try { - return packageManager - .getPackage(eventHandler, label.getPackageIdentifier()) - .getTarget(label.getName()); - } catch (NoSuchPackageException | NoSuchTargetException | InterruptedException e) { - throw new RuntimeException(e); - } - }) - .collect(ImmutableSet.toImmutableSet()); - } - public String getWorkspaceName() { return workspaceName; } @@ -184,7 +156,6 @@ public final class TargetPatternPhaseValue implements SkyValue { TargetPatternPhaseValue that = (TargetPatternPhaseValue) obj; return Objects.equals(this.targetLabels, that.targetLabels) && Objects.equals(this.testsToRunLabels, that.testsToRunLabels) - && Objects.equals(this.removedTargetLabels, that.removedTargetLabels) && Objects.equals(this.workspaceName, that.workspaceName) && this.hasError == that.hasError && this.hasPostExpansionError == that.hasPostExpansionError; @@ -195,7 +166,6 @@ public final class TargetPatternPhaseValue implements SkyValue { return Objects.hash( this.targetLabels, this.testsToRunLabels, - this.removedTargetLabels, this.workspaceName, this.hasError, this.hasPostExpansionError); |