From 5fa4f4f6843fefdc64918cdfb14bf2c287bd387c Mon Sep 17 00:00:00 2001 From: Maksim Denisov Date: Tue, 28 Jan 2025 15:33:52 +0100 Subject: BazelNetwork: Use unordered_set in UploadBlobs instead of BazelBlobContainer --- src/buildtool/execution_api/remote/bazel/bazel_api.cpp | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'src/buildtool/execution_api/remote/bazel/bazel_api.cpp') diff --git a/src/buildtool/execution_api/remote/bazel/bazel_api.cpp b/src/buildtool/execution_api/remote/bazel/bazel_api.cpp index ae5285df..1c8fd96d 100644 --- a/src/buildtool/execution_api/remote/bazel/bazel_api.cpp +++ b/src/buildtool/execution_api/remote/bazel/bazel_api.cpp @@ -141,19 +141,19 @@ namespace { [[nodiscard]] auto ConvertToBazelBlobContainer( ArtifactBlobContainer&& container) noexcept - -> std::optional { - std::vector blobs; + -> std::optional> { + std::unordered_set blobs; try { blobs.reserve(container.Size()); for (const auto& blob : container.Blobs()) { - blobs.emplace_back(ArtifactDigestFactory::ToBazel(blob.digest), - blob.data, - blob.is_exec); + blobs.emplace(ArtifactDigestFactory::ToBazel(blob.digest), + blob.data, + blob.is_exec); } } catch (...) { return std::nullopt; } - return BazelBlobContainer{std::move(blobs)}; + return blobs; } } // namespace @@ -531,7 +531,7 @@ auto BazelApi::CreateAction( bool skip_find_missing) const noexcept -> bool { auto bazel_blobs = ConvertToBazelBlobContainer(std::move(blobs)); - return bazel_blobs ? network_->UploadBlobs(std::move(*bazel_blobs), + return bazel_blobs ? network_->UploadBlobs(*std::move(bazel_blobs), skip_find_missing) : false; } -- cgit v1.2.3