tree 77322244abd7a7f371d0b0807b1f45abcaf5286b
parent ce0b2b23189bc31c135573015ad06748101bc1ae
author chiwang <chiwang@google.com> 1615883705 -0700
committer Copybara-Service <copybara-worker@google.com> 1615883764 -0700

Remote: Don't delete downloaded files for remote build without bytes.

Before this change, when remote build without bytes is enabled, intermediate outputs which are also inputs to local actions will be downloaded for local execution. After build finished, these downloaded files are deleted to keep Bazel's view of the output base identical with the output base i.e. files that Bazel thinks exist only remotely actually do. However, these intermediate outputs maybe used later in which case Bazel need to download them again which is a performance issue. https://github.com/bazelbuild/bazel/issues/12855

This change fix the issue by removing the code used to delete downloaded files. Bazel should be able to take the local file as the source of truth if it exists (otherwise, it is a bug).

This is also an essential step to implement separation of downloads. https://github.com/bazelbuild/bazel/issues/12665

PiperOrigin-RevId: 363131672