diff options
author | shahan <shahan@google.com> | 2018-03-10 05:14:09 -0800 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-03-10 05:15:51 -0800 |
commit | 50f99d56bb710f01524bd3f04882dada16a792c3 (patch) | |
tree | 09e3902e29ebf3a8a7d41db5af4d310989fbe591 /src/main/java/com/google/devtools/build/lib/skyframe/actiongraph/ActionGraphDump.java | |
parent | deccc485603c004daad959fd747f1c0c9efc4f00 (diff) |
ActionOwner references only BuildEvent instead of full BuildConfiguration
This reduces the size of its serialized representation.
PiperOrigin-RevId: 188597127
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/skyframe/actiongraph/ActionGraphDump.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/skyframe/actiongraph/ActionGraphDump.java | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/actiongraph/ActionGraphDump.java b/src/main/java/com/google/devtools/build/lib/skyframe/actiongraph/ActionGraphDump.java index 3a18d414c4..8c68b2655e 100644 --- a/src/main/java/com/google/devtools/build/lib/skyframe/actiongraph/ActionGraphDump.java +++ b/src/main/java/com/google/devtools/build/lib/skyframe/actiongraph/ActionGraphDump.java @@ -28,8 +28,8 @@ import com.google.devtools.build.lib.analysis.AnalysisProtos; import com.google.devtools.build.lib.analysis.AnalysisProtos.ActionGraphContainer; import com.google.devtools.build.lib.analysis.ConfiguredTarget; import com.google.devtools.build.lib.analysis.actions.SpawnAction; -import com.google.devtools.build.lib.analysis.config.BuildConfiguration; import com.google.devtools.build.lib.analysis.configuredtargets.RuleConfiguredTarget; +import com.google.devtools.build.lib.buildeventstream.BuildEvent; import com.google.devtools.build.lib.collect.nestedset.NestedSet; import com.google.devtools.build.lib.collect.nestedset.NestedSetBuilder; import com.google.devtools.build.lib.collect.nestedset.NestedSetView; @@ -103,7 +103,7 @@ public class ActionGraphDump { // store environment if (action instanceof SpawnAction) { SpawnAction spawnAction = (SpawnAction) action; - // TODO(twerth): This handles the fixed environemnt. We probably want to output the inherited + // TODO(twerth): This handles the fixed environment. We probably want to output the inherited // environment as well. ImmutableMap<String, String> fixedEnvironment = spawnAction.getEnvironment(); for (Entry<String, String> environmentVariable : fixedEnvironment.entrySet()) { @@ -122,8 +122,8 @@ public class ActionGraphDump { ActionOwner actionOwner = action.getOwner(); if (actionOwner != null) { - BuildConfiguration buildConfiguration = (BuildConfiguration) actionOwner.getConfiguration(); - actionBuilder.setConfigurationId(knownConfigurations.dataToId(buildConfiguration)); + BuildEvent event = actionOwner.getConfiguration(); + actionBuilder.setConfigurationId(knownConfigurations.dataToId(event)); // store aspect for (AspectDescriptor aspectDescriptor : actionOwner.getAspectDescriptors()) { |