Begin work ensuring that the pool is correctly being destroyed on connection failures

lib/srh/http/command_handler.ex

@@ -169,12 +169,20 @@ defmodule Srh.Http.CommandHandler do
        when is_number(max_connections) do
     case GenRegistry.lookup_or_start(Client, srh_id, [max_connections, connection_info]) do
       {:ok, pid} ->
+        IO.puts("SRH: Found client #{inspect(pid)}")
+
         # Run the command
         case Client.find_worker(pid)
              |> ClientWorker.redis_command(command_array) do
           {:ok, res} ->
             {:ok, %{result: res}}
 
+          # Jedix connection error
+          {:error, %{reason: :closed} = error} ->
+            # Ensure that this pool is killed, but still pass the error up the chain for the response
+            Client.destroy_workers(pid)
+            decode_error(error, srh_id)
+
           {:error, error} ->
             decode_error(error, srh_id)
         end

lib/srh/redis/client.ex

@@ -35,6 +35,10 @@ defmodule Srh.Redis.Client do
     GenServer.cast(client, {:return_worker, pid})
   end
 
+  def destroy_workers(client) do
+    GenServer.cast(client, {:destroy_workers})
+  end
+
   def handle_call({:find_worker}, _from, %{registry_pid: registry_pid} = state)
       when is_pid(registry_pid) do
     {:ok, worker} = ClientRegistry.find_worker(registry_pid)
@@ -59,13 +63,17 @@ defmodule Srh.Redis.Client do
     {:noreply, state}
   end
 
+  def handle_cast({:destroy_workers}, state) do
+    ClientRegistry.destroy_workers(state.registry_pid)
+    {:stop, :normal, state}
+  end
+
   def handle_cast(_msg, state) do
     {:noreply, state}
   end
 
   def handle_info(:idle_death, state) do
     ClientRegistry.destroy_workers(state.registry_pid)
-
     {:stop, :normal, state}
   end
 

lib/srh/redis/client_registry.ex

@@ -33,6 +33,8 @@ defmodule Srh.Redis.ClientRegistry do
   end
 
   def destroy_workers(registry) do
+    # TODO: remove before shipping
+    IO.puts("DEBUG: Destroying workers")
     GenServer.cast(registry, {:destroy_workers})
   end