diff options
author | Klaus Aehlig <klaus.aehlig@huawei.com> | 2025-02-12 17:29:50 +0100 |
---|---|---|
committer | Klaus Aehlig <klaus.aehlig@huawei.com> | 2025-02-12 17:29:50 +0100 |
commit | 51b650e404bf4ff06a0751275fc6fc74ddb23bfd (patch) | |
tree | 20824ec4e29a7c7e33f39206e78ce0a1e2dd9bb1 /src/buildtool/execution_api/remote/bazel | |
parent | ab8de5f08a47cc757f47f30ce03a28d06c8de689 (diff) | |
download | justbuild-51b650e404bf4ff06a0751275fc6fc74ddb23bfd.tar.gz |
BatchReadBlob: fix error reporting
The WithRetry function returns true upon success; therefore a
failure occured if the negation of the value is true. We should
report an error only in case of failure.
Diffstat (limited to 'src/buildtool/execution_api/remote/bazel')
-rw-r--r-- | src/buildtool/execution_api/remote/bazel/bazel_cas_client.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/buildtool/execution_api/remote/bazel/bazel_cas_client.cpp b/src/buildtool/execution_api/remote/bazel/bazel_cas_client.cpp index 8384cdac..e5392c7d 100644 --- a/src/buildtool/execution_api/remote/bazel/bazel_cas_client.cpp +++ b/src/buildtool/execution_api/remote/bazel/bazel_cas_client.cpp @@ -307,7 +307,7 @@ auto BazelCasClient::BatchReadBlobs( }, retry_config_, logger_); - has_failure = has_failure or retry_result; + has_failure = has_failure or not retry_result; } if (has_failure) { logger_.Emit(LogLevel::Error, "Failed to BatchReadBlobs."); |