Encapsulate ExHashRing calls within HashRing

Author: cabol

lib/nebulex/adapters/redis/client_side_cluster/hash_ring.ex

@@ -11,15 +11,33 @@ defmodule Nebulex.Adapters.Redis.ClientSideCluster.HashRing do
 
     @doc false
     defdelegate find_node(ring, key), to: ExHashRing.Ring
+
+    @doc false
+    defdelegate add_node(ring, node_name, replicas), to: ExHashRing.Ring
+
+    @doc false
+    defdelegate get_replicas, to: ExHashRing.Configuration
   else
     @doc false
     def child_spec(_ring_name) do
-      raise ":ex_hash_ring dependency is required to use :client_side_cluster mode"
+      raise err_msg()
     end
 
     @doc false
     def find_node(_ring, _key) do
-      raise ":ex_hash_ring dependency is required to use :client_side_cluster mode"
+      raise err_msg()
     end
+
+    @doc false
+    def add_node(_ring, _node_name, _replicas) do
+      raise err_msg()
+    end
+
+    @doc false
+    def get_replicas do
+      raise err_msg()
+    end
+
+    defp err_msg, do: ":ex_hash_ring dependency is required to use :client_side_cluster mode"
   end
 end

lib/nebulex/adapters/redis/client_side_cluster/node_supervisor.ex

@@ -2,8 +2,7 @@ defmodule Nebulex.Adapters.Redis.ClientSideCluster.NodeSupervisor do
   @moduledoc false
   use Supervisor
 
-  alias ExHashRing.{Configuration, Ring}
-  alias Nebulex.Adapters.Redis.ClientSideCluster.PoolSupervisor
+  alias Nebulex.Adapters.Redis.ClientSideCluster.{HashRing, PoolSupervisor}
 
   ## API
 
@@ -26,10 +25,10 @@ defmodule Nebulex.Adapters.Redis.ClientSideCluster.NodeSupervisor do
 
         {replicas, node_opts} =
           Keyword.pop_lazy(node_opts, :ch_ring_replicas, fn ->
-            Configuration.get_replicas()
+            HashRing.get_replicas()
           end)
 
-        {:ok, _} = Ring.add_node(ring, node_name, replicas)
+        _ignore = HashRing.add_node(ring, node_name, replicas)
 
         node_opts =
           node_opts