diff options
author | Maksim Denisov <denisov.maksim@huawei.com> | 2025-01-28 15:33:52 +0100 |
---|---|---|
committer | Maksim Denisov <denisov.maksim@huawei.com> | 2025-02-07 14:58:04 +0100 |
commit | 5fa4f4f6843fefdc64918cdfb14bf2c287bd387c (patch) | |
tree | 12edde49bfa8676c3e75a10d85cadef9b661088a /src/buildtool/execution_api/remote/bazel/bazel_api.cpp | |
parent | 477c7d5e496b8eb5113b3b51f7152e05eae51fd7 (diff) | |
download | justbuild-5fa4f4f6843fefdc64918cdfb14bf2c287bd387c.tar.gz |
BazelNetwork: Use unordered_set in UploadBlobs instead of BazelBlobContainer
Diffstat (limited to 'src/buildtool/execution_api/remote/bazel/bazel_api.cpp')
-rw-r--r-- | src/buildtool/execution_api/remote/bazel/bazel_api.cpp | 14 |
1 files changed, 7 insertions, 7 deletions
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<BazelBlobContainer> { - std::vector<BazelBlob> blobs; + -> std::optional<std::unordered_set<BazelBlob>> { + std::unordered_set<BazelBlob> 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; } |