From 51f8b186803292f111011d04d5291deb374dc34c Mon Sep 17 00:00:00 2001 From: Oliver Reiche Date: Thu, 6 Oct 2022 12:18:25 +0200 Subject: LocalTreeMap: Prevent tree objects from being stored ... to align with the original idea of caching a flat list of blob objects, without the need to recursively traverse any trees. Consequently, we cannot create any map entry in places where we do not have all sub-tree entries at hand (e.g., LocalAPI, BazelAPI, BazelResponse). --- src/buildtool/execution_api/remote/bazel/bazel_response.hpp | 6 ------ 1 file changed, 6 deletions(-) (limited to 'src/buildtool/execution_api/remote/bazel/bazel_response.hpp') diff --git a/src/buildtool/execution_api/remote/bazel/bazel_response.hpp b/src/buildtool/execution_api/remote/bazel/bazel_response.hpp index 778efa0a..f304210f 100644 --- a/src/buildtool/execution_api/remote/bazel/bazel_response.hpp +++ b/src/buildtool/execution_api/remote/bazel/bazel_response.hpp @@ -47,16 +47,13 @@ class BazelResponse final : public IExecutionResponse { private: std::string action_id_{}; std::shared_ptr const network_{}; - std::shared_ptr const tree_map_{}; BazelExecutionClient::ExecutionOutput output_{}; BazelResponse(std::string action_id, std::shared_ptr network, - std::shared_ptr tree_map, BazelExecutionClient::ExecutionOutput output) : action_id_{std::move(action_id)}, network_{std::move(network)}, - tree_map_{std::move(tree_map)}, output_{std::move(output)} {} [[nodiscard]] auto ReadStringBlob(bazel_re::Digest const& id) noexcept @@ -69,9 +66,6 @@ class BazelResponse final : public IExecutionResponse { [[nodiscard]] auto UploadTreeMessageDirectories( bazel_re::Tree const& tree) const -> std::optional; - - [[nodiscard]] auto ProcessDirectoryMessage(bazel_re::Directory const& dir) - const noexcept -> std::optional; }; #endif // INCLUDED_SRC_BUILDTOOL_EXECUTION_API_REMOTE_BAZEL_BAZEL_RESPONSE_HPP -- cgit v1.2.3