ExecutionApi: Retrieve to correct CAS directly

... by keeping track of each blob being a file or executable
and storing it to the correct local physical CAS directory.
The new flag is merely a hint and only used by the local
execution API. Leaving it out will still correctly transfer
the blob but may cause unnecessary duplicates in file CAS.

1 files changed, 3 insertions, 3 deletions

diff --git a/test/utils/remote_execution/bazel_action_creator.hpp b/test/utils/remote_execution/bazel_action_creator.hpp
index f6ed773a..ae81ae88 100644
--- a/test/utils/remote_execution/bazel_action_creator.hpp
+++ b/test/utils/remote_execution/bazel_action_creator.hpp
@@ -59,12 +59,12 @@
 
     auto cmd_data = cmd.SerializeAsString();
     auto cmd_id = ArtifactDigest::Create<ObjectType::File>(cmd_data);
-    blobs.emplace_back(cmd_id, cmd_data);
+    blobs.emplace_back(cmd_id, cmd_data, /*is_exec=*/false);
 
     bazel_re::Directory empty_dir;
     auto dir_data = empty_dir.SerializeAsString();
     auto dir_id = ArtifactDigest::Create<ObjectType::File>(dir_data);
-    blobs.emplace_back(dir_id, dir_data);
+    blobs.emplace_back(dir_id, dir_data, /*is_exec=*/false);
 
     bazel_re::Action action;
     action.set_allocated_command_digest(
@@ -75,7 +75,7 @@
 
     auto action_data = action.SerializeAsString();
     auto action_id = ArtifactDigest::Create<ObjectType::File>(action_data);
-    blobs.emplace_back(action_id, action_data);
+    blobs.emplace_back(action_id, action_data, /*is_exec=*/false);
 
     BazelCasClient cas_client(info->host, info->port);