diff options
author | mjhalupka <mjhalupka@google.com> | 2018-02-09 09:24:39 -0800 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-02-09 09:26:43 -0800 |
commit | 47f567739920505c120ab0b7775c4cf5e69d6b30 (patch) | |
tree | ad08eae8e1fa8cb9f582e868c22d2ca5956a5620 /src | |
parent | 22d5a658a22d33981e08633ddbef256d05a2c515 (diff) |
Stop populating deprecated fields in the TargetComplete Message in the
TargetCompletedEvent.
PiperOrigin-RevId: 185150827
Diffstat (limited to 'src')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/analysis/TargetCompleteEvent.java | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/TargetCompleteEvent.java b/src/main/java/com/google/devtools/build/lib/analysis/TargetCompleteEvent.java index ecad21968d..607984dc75 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/TargetCompleteEvent.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/TargetCompleteEvent.java @@ -43,7 +43,6 @@ import com.google.devtools.build.lib.collect.nestedset.NestedSetBuilder; import com.google.devtools.build.lib.collect.nestedset.NestedSetView; import com.google.devtools.build.lib.collect.nestedset.Order; import com.google.devtools.build.lib.packages.AttributeMap; -import com.google.devtools.build.lib.packages.TestSize; import com.google.devtools.build.lib.rules.AliasConfiguredTarget; import com.google.devtools.build.lib.syntax.Type; import com.google.devtools.build.skyframe.SkyValue; @@ -155,7 +154,7 @@ public final class TargetCompleteEvent // For tests, announce all the test actions that will minimally happen (except for // interruption). If after the result of a test action another attempt is necessary, // it will be announced with the action that made the new attempt necessary. - Label label = target.getTarget().getLabel(); + Label label = target.getLabel(); TestProvider.TestParams params = target.getProvider(TestProvider.class).getTestParams(); for (int run = 0; run < Math.max(params.getRuns(), 1); run++) { for (int shard = 0; shard < Math.max(params.getShards(), 1); shard++) { @@ -187,16 +186,9 @@ public final class TargetCompleteEvent BuildEventStreamProtos.TargetComplete.newBuilder(); builder.setSuccess(!failed()); - builder.setTargetKind(target.getTarget().getTargetKind()); builder.addAllTag(getTags()); builder.addAllOutputGroup(getOutputFilesByGroup(converters.artifactGroupNamer())); - if (isTest) { - builder.setTestSize( - TargetConfiguredEvent.bepTestSize( - TestSize.getTestSize(target.getTarget().getAssociatedRule()))); - } - // TODO(aehlig): remove direct reporting of artifacts as soon as clients no longer // need it. for (ArtifactsInOutputGroup group : outputs) { |