diff options
| author | Maksim Denisov <denisov.maksim@huawei.com> | 2024-07-31 17:35:48 +0200 |
|---|---|---|
| committer | Maksim Denisov <denisov.maksim@huawei.com> | 2024-08-07 14:43:19 +0200 |
| commit | 74fbd57a3afabc04f594889e092cc735a1788346 (patch) | |
| tree | 44f8768127a7cbc40b06722e9b3416a9a1ebe24e /src/buildtool/execution_api/execution_service/execution_server.hpp | |
| parent | 3188fb1a9b0e967969a3a7953923bcf94247bc09 (diff) | |
| download | justbuild-74fbd57a3afabc04f594889e092cc735a1788346.tar.gz | |
Remove ExecutionServiceImpl::StoreActionResult method
...and move the storing logic to Execute directly. There is no need to pass additional parameters to the method to just perform a check inside, and after this removal there is no need to preserve a one-line method.
Diffstat (limited to 'src/buildtool/execution_api/execution_service/execution_server.hpp')
| -rw-r--r-- | src/buildtool/execution_api/execution_service/execution_server.hpp | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/src/buildtool/execution_api/execution_service/execution_server.hpp b/src/buildtool/execution_api/execution_service/execution_server.hpp index 9ffaa9a7..f1337604 100644 --- a/src/buildtool/execution_api/execution_service/execution_server.hpp +++ b/src/buildtool/execution_api/execution_service/execution_server.hpp @@ -154,13 +154,6 @@ class ExecutionServiceImpl final : public bazel_re::Execution::Service { IExecutionResponse::Ptr const& i_execution_response) const noexcept -> expected<::bazel_re::ExecuteResponse, std::string>; - [[nodiscard]] auto StoreActionResult( - ::bazel_re::ExecuteRequest const* request, - IExecutionResponse::Ptr const& i_execution_response, - ::bazel_re::ExecuteResponse const& execute_response, - ::bazel_re::Action const& action) const noexcept - -> expected<std::monostate, std::string>; - void WriteResponse( ::bazel_re::ExecuteResponse const& execute_response, ::grpc::ServerWriter<::google::longrunning::Operation>* writer, |