Why Supervisor.start_child dont work

Question

I'm beginner in Elixir.

I have one application that initiate one custom supervisor in application.ex. Code:

defmodule MyApp do
  use Application

  def start(_type, _args) do
    import Supervisor.Spec

    children = [
      supervisor(MyApp.Web.Endpoint, []),
      supervisor(MyApp.Repo, []),

      #my notifier
      MyApp.MyNotifier.Supervisor
    ]
    opts = [strategy: :one_for_one, name: MyApp.Supervisor]
    Supervisor.start_link(children, opts)
  end
end

And the code of supervisor is something like this:

defmodule MyApp.MyNotifier.Supervisor do

  use Supervisor

  def start_link(_options), do:
    Supervisor.start_link(__MODULE__, :ok, name: __MODULE__)

  def start_my_notifier(state) do
    Supervisor.start_child(__MODULE__, state)
  end

  def init(:ok) do
    Supervisor.init([], strategy: :one_for_one)
  end

end

And the code of the worker is something like this:

defmodule MyApp.MyNotifier do
  use GenServer

  # Client

  def start_link(state) do
    GenServer.start_link(__MODULE__, state)
  end

  # Server

  def init(state) do
    # Reschedule
    reschedule(state)
    # Reply
    {:ok, state}
  end

  def handle_info(:reschedule, state) do

    case state["count"] < 9 do
      true ->
        # Send notification
        MyNotifier.Helper.notify_past_delivery_time(sate["id"])
        # Reschedule once more
        reschedule(state)
      false ->
        # End process
        Process.exit(self(), :normal)
    end

    {:noreply, state}
  end

  defp reschedule(state) do
    Process.send_after(self(), :reschedule, state["time"] * 60 * 1000)
  end
end

And when something happens in my application i want add/start dynamically one worker with the follow code:

MyNotifier.Supervisor.start_my_notifier(%{"name" => name, "id" => id, "time" => 15, "count" => 0})

When I run my application in debug mode (iex -S mix phx.server) and put one IEx.pry in init function of the worker (then we force the application to go to the state starting the child). Why application never stops?

Answer 1

I changed the supervisor strategy for simple_one_for_one in supervisor code.

Supervisor.init([], strategy: :simple_one_for_one)

This works for me.

Answer 2

The second parameter of call to Supervisor.start_child/2 should be a child spec, so instead of:

Supervisor.start_child(__MODULE__, state)

it should be somewhat like:

Supervisor.start_child(__MODULE__, [self(), MyApp.MyNotifier, [state]])