[Core] Cover cpplint for /src/ray/core_worker (excluding transport) (ray-project#51557)

Signed-off-by: Tatsuya Nishiyama <[email protected]>

Author: nishi-t

.pre-commit-config.yaml

@@ -47,8 +47,12 @@ repos:
     hooks:
       - id: cpplint
         args: ["--filter=-whitespace/braces,-whitespace/line_length,-build/c++11,-build/c++14,-build/c++17,-readability/braces,-whitespace/indent_namespace,-runtime/int,-runtime/references,-build/include_order"]
-        files: ^src/ray/(util|raylet_client|internal|scheduling|pubsub|object_manager|rpc(?:/.*)?|raylet|core_worker/transport)/.*\.(h|cc)$
-        exclude: ^src/ray/raylet/scheduling/.*\.(h|cc)$
+        files: ^src/ray/(util|raylet_client|internal|scheduling|pubsub|object_manager|rpc(?:/.*)?|raylet|core_worker)/.*\.(h|cc)$
+        exclude: |
+          (?x)^(
+            src/ray/raylet/scheduling/.*\.(h|cc)$ |
+            src/ray/core_worker/lib/java/.*\.h$
+          )
 
   - repo: https://github.com/keith/pre-commit-buildifier
     rev: 8.0.1

src/ray/core_worker/actor_creator.h

@@ -13,7 +13,10 @@
 // limitations under the License.
 
 #pragma once
+
 #include <memory>
+#include <utility>
+#include <vector>
 
 #include "ray/common/ray_config.h"
 #include "ray/gcs/gcs_client/gcs_client.h"

src/ray/core_worker/actor_handle.cc

@@ -15,6 +15,8 @@
 #include "ray/core_worker/actor_handle.h"
 
 #include <memory>
+#include <string>
+#include <unordered_map>
 
 namespace ray {
 namespace core {

src/ray/core_worker/actor_handle.h

@@ -16,6 +16,10 @@
 
 #include <gtest/gtest_prod.h>
 
+#include <string>
+#include <unordered_map>
+#include <utility>
+
 #include "absl/types/optional.h"
 #include "ray/common/id.h"
 #include "ray/common/task/task_util.h"
@@ -29,7 +33,7 @@ namespace core {
 
 class ActorHandle {
  public:
-  ActorHandle(rpc::ActorHandle inner) : inner_(std::move(inner)) {}
+  explicit ActorHandle(rpc::ActorHandle inner) : inner_(std::move(inner)) {}
 
   // Constructs a new ActorHandle as part of the actor creation process.
   ActorHandle(const ActorID &actor_id,

src/ray/core_worker/actor_manager.cc

@@ -14,6 +14,11 @@
 
 #include "ray/core_worker/actor_manager.h"
 
+#include <memory>
+#include <string>
+#include <utility>
+#include <vector>
+
 #include "ray/gcs/pb_util.h"
 
 namespace ray {

src/ray/core_worker/actor_manager.h

@@ -16,7 +16,10 @@
 
 #include <gtest/gtest_prod.h>
 
+#include <memory>
+#include <string>
 #include <utility>
+#include <vector>
 
 #include "absl/container/flat_hash_map.h"
 #include "ray/core_worker/actor_creator.h"

src/ray/core_worker/common.cc

@@ -14,6 +14,12 @@
 
 #include "ray/core_worker/common.h"
 
+#include <memory>
+#include <string>
+#include <unordered_map>
+#include <utility>
+#include <vector>
+
 namespace ray {
 namespace core {
 

src/ray/core_worker/common.h

@@ -14,7 +14,11 @@
 
 #pragma once
 
+#include <memory>
 #include <string>
+#include <unordered_map>
+#include <utility>
+#include <vector>
 
 #include "ray/common/id.h"
 #include "ray/common/ray_object.h"

src/ray/core_worker/context.cc

@@ -16,6 +16,9 @@
 
 #include <google/protobuf/util/json_util.h>
 
+#include <memory>
+#include <string>
+
 #include "ray/common/runtime_env_common.h"
 
 namespace ray {

src/ray/core_worker/context.h

@@ -16,6 +16,7 @@
 
 #include <boost/thread.hpp>
 #include <memory>
+#include <string>
 
 #include "absl/base/thread_annotations.h"
 #include "absl/synchronization/mutex.h"