[core] loosen the check on release object (#39570) (#39577)

This check is probably too strict, as the same client might call release object multiple times. This is a benign behavior and we shouldn't crash.

Author: scv119

src/ray/object_manager/plasma/store.cc

@@ -278,9 +278,10 @@ int PlasmaStore::RemoveFromClientObjectIds(const ObjectID &object_id,
 void PlasmaStore::ReleaseObject(const ObjectID &object_id,
                                 const std::shared_ptr<Client> &client) {
   auto entry = object_lifecycle_mgr_.GetObject(object_id);
-  RAY_CHECK(entry != nullptr);
-  // Remove the client from the object's array of clients.
-  RAY_CHECK(RemoveFromClientObjectIds(object_id, client) == 1);
+  if (entry != nullptr) {
+    // Remove the client from the object's array of clients.
+    RemoveFromClientObjectIds(object_id, client);
+  }
 }
 
 void PlasmaStore::SealObjects(const std::vector<ObjectID> &object_ids) {

src/ray/object_manager/plasma/store.h

@@ -175,6 +175,7 @@ class PlasmaStore {
       ABSL_EXCLUSIVE_LOCKS_REQUIRED(mutex_);
 
   /// Record the fact that a particular client is no longer using an object.
+  /// This function is idempotent thus can be called multiple times.
   ///
   /// \param object_id The object ID of the object that is being released.
   /// \param client The client making this request.