diff options
author | Klaus Aehlig <klaus.aehlig@huawei.com> | 2025-06-02 11:42:14 +0200 |
---|---|---|
committer | Klaus Aehlig <klaus.aehlig@huawei.com> | 2025-06-02 14:31:37 +0200 |
commit | d1dc5cce32ba20c0dae13aecf63e88fe8ead899f (patch) | |
tree | 0d37773d8148bb35e037fc5ad2bcc03729903e32 /src/buildtool | |
parent | b4461efba988de9f50b7fac00df5a3592c09a38e (diff) | |
download | justbuild-d1dc5cce32ba20c0dae13aecf63e88fe8ead899f.tar.gz |
ParallelRetrieveToCasWithCache: abort early on failure
... and do mark artifacts internally as synchronized. First all all,
we will abort anyway, to the entry won't even be read and, secondly
it is not necessarily true that the artifact is synchronized.
Diffstat (limited to 'src/buildtool')
-rw-r--r-- | src/buildtool/execution_api/remote/bazel/bazel_api.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/buildtool/execution_api/remote/bazel/bazel_api.cpp b/src/buildtool/execution_api/remote/bazel/bazel_api.cpp index f15d421d..c90b7024 100644 --- a/src/buildtool/execution_api/remote/bazel/bazel_api.cpp +++ b/src/buildtool/execution_api/remote/bazel/bazel_api.cpp @@ -450,6 +450,10 @@ auto BazelApi::CreateAction( return false; } + if (failure) { + return false; + } + try { for (auto const& info : artifacts_info) { done->insert(info); @@ -461,7 +465,7 @@ auto BazelApi::CreateAction( ex.what()); } - return not failure; + return true; } [[nodiscard]] auto BazelApi::RetrieveToMemory( |