diff options
author | Maksim Denisov <denisov.maksim@huawei.com> | 2025-01-29 14:34:36 +0100 |
---|---|---|
committer | Maksim Denisov <denisov.maksim@huawei.com> | 2025-02-07 14:58:04 +0100 |
commit | c30cb3221989b60cee96287a1cb9d9e87f9ba464 (patch) | |
tree | 9d55f69902c511f33d049123acd9815fe0fecfda /src/buildtool/execution_api/remote/bazel/bazel_api.cpp | |
parent | 686a13142b816dfa334bb85f3ffdcc62a81a3bb4 (diff) | |
download | justbuild-c30cb3221989b60cee96287a1cb9d9e87f9ba464.tar.gz |
BazelNetwork: Use ArtifactBlobs in UploadBlobs
Diffstat (limited to 'src/buildtool/execution_api/remote/bazel/bazel_api.cpp')
-rw-r--r-- | src/buildtool/execution_api/remote/bazel/bazel_api.cpp | 23 |
1 files changed, 1 insertions, 22 deletions
diff --git a/src/buildtool/execution_api/remote/bazel/bazel_api.cpp b/src/buildtool/execution_api/remote/bazel/bazel_api.cpp index bee70424..d46c9e09 100644 --- a/src/buildtool/execution_api/remote/bazel/bazel_api.cpp +++ b/src/buildtool/execution_api/remote/bazel/bazel_api.cpp @@ -31,7 +31,6 @@ #include "src/buildtool/common/artifact_digest_factory.hpp" #include "src/buildtool/common/bazel_types.hpp" #include "src/buildtool/common/protocol_traits.hpp" -#include "src/buildtool/execution_api/bazel_msg/bazel_blob_container.hpp" #include "src/buildtool/execution_api/bazel_msg/bazel_common.hpp" #include "src/buildtool/execution_api/bazel_msg/directory_tree.hpp" #include "src/buildtool/execution_api/common/artifact_blob_container.hpp" @@ -138,23 +137,6 @@ namespace { return true; } -[[nodiscard]] auto ConvertToBazelBlobContainer( - std::unordered_set<ArtifactBlob>&& container) noexcept - -> std::optional<std::unordered_set<BazelBlob>> { - std::unordered_set<BazelBlob> blobs; - try { - blobs.reserve(container.size()); - for (const auto& blob : container) { - blobs.emplace(ArtifactDigestFactory::ToBazel(blob.digest), - blob.data, - blob.is_exec); - } - } catch (...) { - return std::nullopt; - } - return blobs; -} - } // namespace BazelApi::BazelApi( @@ -529,10 +511,7 @@ auto BazelApi::CreateAction( [[nodiscard]] auto BazelApi::Upload(std::unordered_set<ArtifactBlob>&& blobs, bool skip_find_missing) const noexcept -> bool { - auto bazel_blobs = ConvertToBazelBlobContainer(std::move(blobs)); - return bazel_blobs ? network_->UploadBlobs(*std::move(bazel_blobs), - skip_find_missing) - : false; + return network_->UploadBlobs(std::move(blobs), skip_find_missing); } [[nodiscard]] auto BazelApi::UploadTree( |