tree 55e45b92722af56d89bdac89b0445727a11303b7
parent b2fac7457a673a13427861aabb323a32b74e07ce
author janakr <janakr@google.com> 1559163436 -0700
committer Copybara-Service <copybara-worker@google.com> 1559163486 -0700

Put ActionLookupData inside DerivedArtifact, and move ArtifactOwner into SourceArtifact.

ActionLookupData is set inside a DerivedArtifact at ConfiguredTarget creation time. In particular, it is now mutable. This means that it is no longer safe to call #getArtifactOwner, #getOwnerLabel, etc. on an artifact if it was created by the configured target that is currently being analyzed.

This allows all callers that were indirectly getting the ActionLookupData via retrieving an ActionLookupKey from the graph to just use it directly. For now, don't actually do very much with this new power. In particular, don't change ArtifactFunction to avoid that lookup.

In a follow-up, ActionExecutionFunction will stop requesting ArtifactValues for normal output artifacts, instead requesting ActionExecutionValues directly, thus avoiding the memory and CPU of creating ArtifactValues for the vast majority of artifacts.

Will also remove ArtifactOwner from ArtifactFactory#getDerivedArtifact in a follow-up.

PiperOrigin-RevId: 250560440