Revert "[core] Adding a private core debugging API to get the internal state of the ReferenceCounter as a JSON string." (#60818)

Reverts #60760

Author: aslonnie

python/ray/_private/debug_api.py

@@ -4616,17 +4616,6 @@ cdef class CoreWorker:
 
         return ref_counts
 
-    def get_reference_counter_debug_json(self):
-        """Returns a JSON string of the internal state of the ReferenceCounter.
-
-        NOTE: This is NOT a stable API. It should only be used for debugging and
-        NEVER in tests or production code.
-        """
-        cdef:
-            c_string debug_json
-        debug_json = CCoreWorkerProcess.GetCoreWorker().GetReferenceCounterDebugJson()
-        return debug_json.decode('utf-8')
-
     def set_get_async_callback(self, ObjectRef object_ref, user_callback: Callable):
         # NOTE: we need to manually increment the Python reference count to avoid the
         # callback object being garbage collected before it's called by the core worker.

python/ray/_raylet.pyx

@@ -328,7 +328,6 @@ cdef extern from "ray/core_worker/core_worker.h" nogil:
         void YieldCurrentFiber(CFiberEvent &coroutine_done)
 
         unordered_map[CObjectID, pair[size_t, size_t]] GetAllReferenceCounts()
-        c_string GetReferenceCounterDebugJson() const
         c_vector[CTaskID] GetPendingChildrenTasks(const CTaskID &task_id) const
 
         void GetAsync(const CObjectID &object_id,

python/ray/includes/libcoreworker.pxd

@@ -169,7 +169,6 @@ ray_cc_library(
         "//src/ray/util:network_util",
         "@com_google_absl//absl/base:core_headers",
         "@com_google_absl//absl/synchronization",
-        "@nlohmann_json",
     ],
 )
 

src/ray/core_worker/BUILD.bazel

@@ -856,10 +856,6 @@ CoreWorker::GetAllReferenceCounts() const {
   return counts;
 }
 
-std::string CoreWorker::GetReferenceCounterDebugJson() const {
-  return reference_counter_->ToJsonString();
-}
-
 std::vector<TaskID> CoreWorker::GetPendingChildrenTasks(const TaskID &task_id) const {
   return task_manager_->GetPendingChildrenTasks(task_id);
 }

src/ray/core_worker/core_worker.cc

@@ -398,12 +398,6 @@ class CoreWorker : public std::enable_shared_from_this<CoreWorker> {
   /// (local, submitted_task) reference counts. For debugging purposes.
   std::unordered_map<ObjectID, std::pair<size_t, size_t>> GetAllReferenceCounts() const;
 
-  /// Returns a JSON string representation of the internal state of the
-  /// ReferenceCounter.
-  /// NOTE: This is very expensive and must only be used for debugging.
-  /// Please do NOT use this for production observability or testing.
-  std::string GetReferenceCounterDebugJson() const;
-
   /// Return all pending children task ids for a given parent task id.
   /// The parent task id should exist in the current worker.
   /// For debugging and testing only.

src/ray/core_worker/core_worker.h

@@ -15,7 +15,6 @@
 #include "ray/core_worker/reference_counter.h"
 
 #include <memory>
-#include <nlohmann/json.hpp>
 #include <string>
 #include <unordered_map>
 #include <unordered_set>
@@ -25,52 +24,6 @@
 #include "ray/util/logging.h"
 #include "ray/util/network_util.h"
 
-using json = nlohmann::json;
-
-namespace {
-
-json AddressToJson(const ray::rpc::Address &address) {
-  return {
-      {"node_id", ray::NodeID::FromBinary(address.node_id()).Hex()},
-      {"ip_address", address.ip_address()},
-      {"port", address.port()},
-      {"worker_id", ray::WorkerID::FromBinary(address.worker_id()).Hex()},
-  };
-}
-
-template <class Container>
-json IdContainerToJsonArray(const Container &c) {
-  json output = json::array();
-  for (const auto &id : c) {
-    output.push_back(id.Hex());
-  }
-  return output;
-}
-
-constexpr const char *LineageReconstructionEligibilityToString(
-    ray::core::LineageReconstructionEligibility lre) noexcept {
-  switch (lre) {
-  case ray::core::LineageReconstructionEligibility::ELIGIBLE:
-    return "ELIGIBLE";
-  case ray::core::LineageReconstructionEligibility::INELIGIBLE_PUT:
-    return "INELIGIBLE_PUT";
-  case ray::core::LineageReconstructionEligibility::INELIGIBLE_NO_RETRIES:
-    return "INELIGIBLE_NO_RETRIES";
-  case ray::core::LineageReconstructionEligibility::INELIGIBLE_LOCAL_MODE:
-    return "INELIGIBLE_LOCAL_MODE";
-  case ray::core::LineageReconstructionEligibility::INELIGIBLE_LINEAGE_EVICTED:
-    return "INELIGIBLE_LINEAGE_EVICTED";
-  case ray::core::LineageReconstructionEligibility::INELIGIBLE_LINEAGE_DISABLED:
-    return "INELIGIBLE_LINEAGE_DISABLED";
-  case ray::core::LineageReconstructionEligibility::INELIGIBLE_REF_NOT_FOUND:
-    return "INELIGIBLE_REF_NOT_FOUND";
-  default:
-    return "UNKNOWN";
-  };
-};
-
-};  // namespace
-
 #define PRINT_REF_COUNT(it) \
   RAY_LOG(DEBUG) << "REF " << it->first << ": " << it->second.DebugString();
 
@@ -1813,76 +1766,6 @@ std::string ReferenceCounter::DebugString() const {
   return ss.str();
 }
 
-json ReferenceCounter::NestedReferenceCount::ToJson() const {
-  return {
-      {"contained_in_owned", IdContainerToJsonArray(contained_in_owned)},
-      {"contained_in_borrowed_ids", IdContainerToJsonArray(contained_in_borrowed_ids)},
-      {"contains", IdContainerToJsonArray(contains)}};
-}
-
-json ReferenceCounter::BorrowInfo::ToJson() const {
-  json stored_in_objects_json = json::array();
-  for (const auto &[object_id, addr] : stored_in_objects) {
-    stored_in_objects_json.push_back(
-        {{"object_id", object_id.Hex()}, {"address", AddressToJson(addr)}});
-  }
-  json borrowers_json = json::array();
-  for (const auto &address : borrowers) {
-    borrowers_json.push_back(AddressToJson(address));
-  }
-  return {{"stored_in_objects", stored_in_objects_json}, {"borrowers", borrowers_json}};
-}
-
-std::string ReferenceCounter::ToJsonString() const {
-  absl::MutexLock lock(&mutex_);
-  json ref_table_json = json::array();
-  for (const auto &[obj_id, reference] : object_id_refs_) {
-    ref_table_json.push_back({
-        {"object_id", obj_id.Hex()},
-        {"reference", reference.ToJson()},
-    });
-  }
-  json output = {{"rpc_address", AddressToJson(rpc_address_)},
-                 {"reference_table", ref_table_json},
-                 {"freed_objects", IdContainerToJsonArray(freed_objects_)},
-                 {"reconstructable_owned_objects",
-                  IdContainerToJsonArray(reconstructable_owned_objects_)},
-                 {"objects_to_recover", IdContainerToJsonArray(objects_to_recover_)}};
-  return output.dump();
-}
-
-json ReferenceCounter::Reference::ToJson() const {
-  return {
-      {"call_site", call_site_},
-      {"object_size", object_size_},
-      {"locations", IdContainerToJsonArray(locations)},
-      {"owner_address", owner_address_ ? AddressToJson(*owner_address_) : json(nullptr)},
-      {"pinned_at_node_id",
-       pinned_at_node_id_ ? json(pinned_at_node_id_->Hex()) : json(nullptr)},
-      {"tensor_transport", tensor_transport_ ? json(*tensor_transport_) : json(nullptr)},
-      {"owned_by_us", owned_by_us_},
-      {"lineage_eligibility",
-       LineageReconstructionEligibilityToString(lineage_eligibility_)},
-      {"lineage_ref_count", lineage_ref_count},
-      {"local_ref_count", local_ref_count},
-      {"submitted_task_ref_count", submitted_task_ref_count},
-      {"nested_reference_count",
-       nested_reference_count ? nested_reference_count->ToJson() : json(nullptr)},
-      {"borrow_info", borrow_info ? borrow_info->ToJson() : json(nullptr)},
-      {"num_object_out_of_scope_or_freed_callbacks",
-       on_object_out_of_scope_or_freed_callbacks.size()},
-      {"num_object_ref_deleted_callbacks", object_ref_deleted_callbacks.size()},
-      {"publish_ref_removed", publish_ref_removed},
-      {"spilled_url", spilled_url},
-      {"spilled_node_id", spilled_node_id.Hex()},
-      {"spilled", spilled},
-      {"foreign_owner_already_monitoring", foreign_owner_already_monitoring},
-      {"has_nested_refs_report", has_nested_refs_to_report},
-      {"pending_creation", pending_creation_},
-      {"did_spill", did_spill},
-  };
-}
-
 std::string ReferenceCounter::Reference::DebugString() const {
   std::stringstream ss;
   ss << "Reference{borrowers: " << borrow().borrowers.size()

src/ray/core_worker/reference_counter.cc

@@ -16,7 +16,6 @@
 
 #include <list>
 #include <memory>
-#include <nlohmann/json.hpp>
 #include <string>
 #include <unordered_map>
 #include <unordered_set>
@@ -183,8 +182,6 @@ class ReferenceCounter : public ReferenceCounterInterface,
 
   std::string DebugString() const override ABSL_LOCKS_EXCLUDED(mutex_);
 
-  std::string ToJsonString() const override ABSL_LOCKS_EXCLUDED(mutex_);
-
   void PopAndClearLocalBorrowers(const std::vector<ObjectID> &borrowed_ids,
                                  ReferenceTableProto *proto,
                                  std::vector<ObjectID> *deleted) override
@@ -282,8 +279,6 @@ class ReferenceCounter : public ReferenceCounterInterface,
     ///  2. We call ray.get() on an ID whose contents we do not know and we
     ///     discover that it contains these IDs.
     absl::flat_hash_set<ObjectID> contains;
-
-    nlohmann::json ToJson() const;
   };
 
   /// Contains information related to borrowing only.
@@ -307,8 +302,6 @@ class ReferenceCounter : public ReferenceCounterInterface,
     ///     borrowers. A borrower is removed from the list when it responds
     ///     that it is no longer using the reference.
     absl::flat_hash_set<rpc::Address> borrowers;
-
-    nlohmann::json ToJson() const;
   };
 
   struct Reference {
@@ -429,7 +422,6 @@ class ReferenceCounter : public ReferenceCounterInterface,
     }
 
     std::string DebugString() const;
-    nlohmann::json ToJson() const;
 
     /// Description of the call site where the reference was created.
     std::string call_site_ = "<unknown>";

src/ray/core_worker/reference_counter.h

@@ -387,11 +387,6 @@ class ReferenceCounterInterface {
 
   virtual std::string DebugString() const = 0;
 
-  /// Converts the entire private state into a JSON string.
-  /// NOTE: This is a very expensive method. It is only available for debugging.
-  /// Do not use it for production observability or testing.
-  virtual std::string ToJsonString() const = 0;
-
   /// Populate a table with ObjectIDs that we were or are still borrowing.
   /// This should be called when a task returns, and the argument should be any
   /// IDs that were passed by reference in the task spec or that were