willhains / guicebox

Automatically exported from code.google.com/p/guicebox
2 stars 2 forks source link

Can't kill GuiceBox process #21

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Steps to reproduce:
1. Create a simple GuiceBox app with the code below.
{{{
    public static void main(String[] args)
    {
        final Injector injector = Guice.createInjector(new CommandLineModule(), new 
UdpFailoverModule("sample.empty"));
        final GuiceBox guicebox = injector.getInstance(GuiceBox.class);
        guicebox.start();
    }
}}}
2. Run the app with appropriate properties for UdpFailover.
3. Attempt to kill the process (from Eclipse).

Expected result:
The process should die immediately.

Actual result:
The process cannot be killed from Eclipse, but can be killed from 
ActivityMonitor or Terminal.

Original issue reported on code.google.com by willhains on 28 Dec 2008 at 7:17

GoogleCodeExporter commented 9 years ago
Can someone verify this?

Original comment by willhains on 29 Dec 2008 at 8:06

GoogleCodeExporter commented 9 years ago
Confirmed - this is a problem when running from the command line also.

Original comment by willhains on 13 Oct 2009 at 1:52

GoogleCodeExporter commented 9 years ago
Appears to only affect GuiceBox with UdpFailover...

Original comment by willhains on 13 Oct 2009 at 2:21

GoogleCodeExporter commented 9 years ago
Fixed in r149.
Somehow the awaitTermination() method was blocking forever, and not return 
true. I just removed the code that 
waits for the NonBlockingHeart to die.

Original comment by willhains on 13 Oct 2009 at 3:53