bazel
bazel copied to clipboard
Wire up `internal-inline-outputs` requirement with RE API
In-memory outputs marked with the internal-inline-outputs execution requirement are now added to the inline_output_files hint field of the remote execution and remote cache API, potentially saving a round trip on files that are unconditionally read by Bazel client after spawn execution.
Support for output file inlining has been added to the remote worker implementation.
Work towards #8421
Currently stacked on https://github.com/bazelbuild/bazel/pull/23421, but otherwise ready for review.
FYI, the changes to remote_execution.proto must be sent in a separate PR. I can do that tomorrow if you don't beat me to it.
@tjgq I sent https://github.com/bazelbuild/bazel/pull/23458
@tjgq I updated this to say "Fixes #8421" as it realizes inlining for the cases that were originally anticipated. I think it's better to open a dedicated issue for the potential extension to arbitrary outputs.
@tjgq I updated this to say "Fixes #8421" as it realizes inlining for the cases that were originally anticipated. I think it's better to open a dedicated issue for the potential extension to arbitrary outputs.
Per the recent discussion in that issue, I agree.
@bazel-io fork 7.4.0
@tjgq @fmeum can you please resolve the conflicts?
@tjgq @fmeum can you please resolve the conflicts?
@tjgq @fmeum Friendly ping :)
I will do the import and resolve the conflicts.