Should quit_excess_workers_for() respect TERM_CHILD setting?

[iceberg901]

Is it necessary to send a different signal in quit_excess_workers_for when TERM_CHILD=1 is set to ensure clean shutdown of workers under Heroku signal handling conditions? Currently it always sends QUIT

    def quit_excess_workers_for(queues)
      delta = -worker_delta_for(queues)
      pids_for(queues)[0...delta].each do |pid|
        Process.kill("QUIT", pid)
      end
    end

[nevans]

That sounds very reasonable to me.

It's certainly the approach used by graceful_worker_shutdown_and_wait! and graceful_worker_shutdown!, although shutdown_everything_now! is reversed (which is confusing, but based on the default CLI options which will use graceful_worker_shutdown_and_wait when TERM_CHILD is enabled).

Send a small PR, and I'll merge it. :)