diff options
| author | Maksim Denisov <denisov.maksim@huawei.com> | 2024-06-20 16:56:25 +0200 |
|---|---|---|
| committer | Maksim Denisov <denisov.maksim@huawei.com> | 2024-06-25 13:59:48 +0200 |
| commit | e5ca9113797a23824dd4545d93221bd6379dd692 (patch) | |
| tree | aa9ee982e00dada9c7e7151deb016f4fbbcee071 /src/buildtool/execution_api/execution_service | |
| parent | 97c43e0c768cea5486c38881cef4b0d7a4a213f5 (diff) | |
| download | justbuild-e5ca9113797a23824dd4545d93221bd6379dd692.tar.gz | |
Pass IExecutionApi to ServerImpl, Archive by reference
...instead of not_null const ptr.
Diffstat (limited to 'src/buildtool/execution_api/execution_service')
| -rw-r--r-- | src/buildtool/execution_api/execution_service/execution_server.cpp | 2 | ||||
| -rw-r--r-- | src/buildtool/execution_api/execution_service/execution_server.hpp | 6 |
2 files changed, 4 insertions, 4 deletions
diff --git a/src/buildtool/execution_api/execution_service/execution_server.cpp b/src/buildtool/execution_api/execution_service/execution_server.cpp index c3802ea7..b71380f5 100644 --- a/src/buildtool/execution_api/execution_service/execution_server.cpp +++ b/src/buildtool/execution_api/execution_service/execution_server.cpp @@ -133,7 +133,7 @@ auto ExecutionServiceImpl::GetIExecutionAction( auto env_vars = GetEnvVars(*c); - auto i_execution_action = api_->CreateAction( + auto i_execution_action = api_.CreateAction( ArtifactDigest{action.input_root_digest()}, {c->arguments().begin(), c->arguments().end()}, {c->output_files().begin(), c->output_files().end()}, diff --git a/src/buildtool/execution_api/execution_service/execution_server.hpp b/src/buildtool/execution_api/execution_service/execution_server.hpp index 48f50624..98223938 100644 --- a/src/buildtool/execution_api/execution_service/execution_server.hpp +++ b/src/buildtool/execution_api/execution_service/execution_server.hpp @@ -25,8 +25,8 @@ class ExecutionServiceImpl final : public bazel_re::Execution::Service { public: explicit ExecutionServiceImpl( - gsl::not_null<IExecutionApi*> const& local_api) noexcept - : api_{local_api} {} + gsl::not_null<IExecutionApi const*> const& local_api) noexcept + : api_{*local_api} {} // Execute an action remotely. // // In order to execute an action, the client must first upload all of the @@ -111,7 +111,7 @@ class ExecutionServiceImpl final : public bazel_re::Execution::Service { private: gsl::not_null<Storage const*> storage_ = &Storage::Instance(); - gsl::not_null<IExecutionApi*> const api_; + IExecutionApi const& api_; Logger logger_{"execution-service"}; [[nodiscard]] auto GetAction(::bazel_re::ExecuteRequest const* request) |