DRb.stop_service doesn't kill sleeping TimerIdConv threads
According to this commit, it's a good idea for DRb.stop_service to kill threads it creates.
My problem is I have a workflow that creates a new DRb server and provides it a TimerIdConv for keeping drb objects alive until it's not accessed for X seconds. It then stops the service.
This workflow can happen many times.
The TimerIdConv.new(5) call creates a new thread each time that never exits.
There isn't a public interface to kill these threads so I have to use instance variables to get the thread so it can be killed.
Also, you can't reuse the TimerIdConv because you don't know how long it has been sleeping and there is not a way to reset it to ensure it sleeps the full timeout.
I would prefer if I could tell DRb.start_service to use a :timer_idconv => 5, have it create the TimerIdConv and be responsible for killing it when I call stop_service.
See code below. I have tested with ruby 2.3.1 and the code below slowly continues to grow in thread count.
 Commit: c474ecb0dfe0dd0d5e6b2b41f09eaf251d0c7079 or https://github.com/ruby/ruby/commit/c474ecb0dfe0dd0d5e6b2b41f09eaf251d0c7079#diff-140ee364bff64098bb12e0e96fadb402
require 'drb' require 'drb/timeridconv' 100.times do # setup the drb server with a one second timer # creates a new thread that never dies timer = DRb::TimerIdConv.new(5) service = DRb.start_service("druby://127.0.0.1:0", nil, :idconv => timer) # ... some code that uses the DRb service # stop the drb server now that we're done with it service.stop_service # Below is ugly code that kills the timer thread. Without the code below, the # thread stays alive forever. This should have a public API. # It would be nice if DRb.start_service # accepted a :timer_idconv => 5 option that would create the timer thread itself # and was responsible for killing this thread when calling stop_service. # holder = timer.instance_variable_get(:@holder) # keeper = holder.instance_variable_get(:@keeper) if holder # keeper.kill if keeper puts Thread.list.length sleep 2 end
Updated by jrafanie (Joe Rafaniello) over 4 years ago
I have opened a pull request to expose an API for killing the timer thread. This PR should be enough to workaround this issue for me, although it would be nice if DRb.start_service could be responsible for creating/starting the timer and stop_service could then automatically stop any timer thread created by the service: