lint, comments

cpp/include/ray/api.h

@@ -111,8 +111,7 @@ ray::internal::ActorCreator<PyActorClass> Actor(PyActorClass func);
 ray::internal::ActorCreator<JavaActorClass> Actor(JavaActorClass func);
 
 /// Get a handle to a named actor in current namespace.
-/// Gets a handle to a named actor with the given name. The actor must have been created
-/// with name specified.
+/// The actor must have been created with name specified.
 ///
 /// \param[in] actor_name The name of the named actor.
 /// \return An ActorHandle to the actor if the actor of specified name exists or an
@@ -121,8 +120,7 @@ template <typename T>
 boost::optional<ActorHandle<T>> GetActor(const std::string &actor_name);
 
 /// Get a handle to a named actor in the given namespace.
-/// Gets a handle to a named actor with the given name of the given namespace. The actor
-/// must have been created with name specified.
+/// The actor must have been created with name specified.
 ///
 /// \param[in] actor_name The name of the named actor.
 /// \param[in] namespace The namespace of the actor.

cpp/include/ray/api/ray_config.h

@@ -60,6 +60,7 @@ class RayConfig {
  // A specific flag for internal `default_worker`. Please don't use it in user code.
  bool is_worker_ = false;
 
+ // A namespace is a logical grouping of jobs and named actors.
  std::string ray_namespace = "";
 };
 

cpp/src/ray/config_internal.cc

@@ -188,7 +188,7 @@ void ConfigInternal::Init(RayConfig &config, int argc, char **argv) {
  }
  if (worker_type == WorkerType::DRIVER) {
  ray_namespace =
- config.ray_namespace.empty() ? generate_uuid_v4() : config.ray_namespace;
+ config.ray_namespace.empty() ? GenerateUUIDV4() : config.ray_namespace;
  }
 };
 

python/ray/_private/worker.py

@@ -1092,7 +1092,7 @@ def init(
  is true.
  log_to_driver: If true, the output from all of the worker
  processes on all nodes will be directed to the driver.
- namespace: Namespace to use
+ namespace: A namespace is a logical grouping of jobs and named actors.
  runtime_env: The runtime environment to use
  for this job (see :ref:`runtime-environments` for details).
  storage: [Experimental] Specify a URI for persistent cluster-wide storage.

src/ray/util/util.h

@@ -117,12 +117,12 @@ inline int64_t current_sys_time_us() {
  return mu_since_epoch.count();
 }
 
-static std::random_device rd;
-static std::mt19937 gen(rd());
-static std::uniform_int_distribution<> dis(0, 15);
-static std::uniform_int_distribution<> dis2(8, 11);
+inline std::string GenerateUUIDV4() {
+ static std::random_device rd;
+ static std::mt19937 gen(rd());
+ static std::uniform_int_distribution<> dis(0, 15);
+ static std::uniform_int_distribution<> dis2(8, 11);
 
-inline std::string generate_uuid_v4() {
  std::stringstream ss;
  int i;
  ss << std::hex;