无法停止使用ExecutorService启动的任务

时间:2011-12-26 15:10:14

标签: java concurrency executorservice

抱歉,我必须打开一个新线程来描述这个问题。

今天早上我问this question,有一些回复,但我的问题仍未解决。

这次我将附加一些可运行的代码(简化但有相同的问题),以便您重现问题:

public class ThreadPoolTest {
    public static void main(String[] args) throws Exception {
        final ExecutorService taskExecutor = Executors.newFixedThreadPool(5);
        Future<Void> futures[] = new Future[5];
        for (int i = 0; i < futures.length; ++i)
            futures[i] = startTask(taskExecutor);

        for (int i = 0; i < futures.length; ++i)
            System.out.println("futures[i].cancel(true): " + futures[i].cancel(true));

        System.out.println("Cancel DONE.");
        taskExecutor.shutdown();
    }

    private static Future<Void> startTask(final ExecutorService taskExecutor) {
        Future<Void> f = taskExecutor.submit(new Callable<Void>() {
            public Void call() throws Exception {
                try {
                    downloadFile(new URI("http://stackoverflow.com"));
                    while(true) {
                        System.out.println(Thread.currentThread().getName() + ": " + Thread.currentThread().isInterrupted());
                        if(Thread.currentThread().isInterrupted())
                            break;
                    }
                } catch (Exception ex) {
                    ex.printStackTrace();
                }
                return null;
            } 
        });
        return f;
    }

    private static void downloadFile (final URI uri) throws Exception {
//        if(true) return;
        Socket socket = new Socket (uri.getHost(), uri.getPort() == -1 ? 80 : uri.getPort());
        return;
    }
}

上面的代码很可能被困在一个无限循环中(你可能希望多次运行代码来见证我看到的内容),正如你在主要方法中看到的那样我称之为期货[i] .cancel(true)对于所有任务,我不知道为什么会这样,这已经让我折磨了一天多。

非常感谢您的帮助。

3 个答案:

答案 0 :(得分:6)

我玩过你的代码,并注意到在创建套接字之前线程的中断状态有时是真的,之后是假的。

我试过打断一个线程并调用Socket构造函数,然后线程总是保持中断状态。我也尝试删除线程池的关闭,问题继续发生。

然后我尝试使用5种不同的URI,而不是总是使用相同的URI。问题从未发生过。

所以我编写了这个简单的程序,显示线程池不是罪魁祸首,但套接字是:

public static void main(String[] args) throws Exception {
    final URI uri = new URI("http://stackoverflow.com");
    for (int i = 0; i < 5; i++) {
        Runnable r = new Runnable() {
            @Override
            public void run() {
                Thread.currentThread().interrupt();
                System.out.println(Thread.currentThread().isInterrupted());
                try {
                    Socket socket = new Socket (uri.getHost(), uri.getPort() == -1 ? 80 : uri.getPort());
                }
                catch (IOException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }
                System.out.println(Thread.currentThread().isInterrupted());
            }
        };
        new Thread(r).start();
    }
}

实际上,当5个线程为同一个主机和端口创建一个套接字时,其中4个线程的中断状态被清除。

然后我尝试同步套接字创建(在单个锁上,但我猜你可能每个主机/端口使用一个锁):

synchronized(lock) {
    try {
        Socket socket = new Socket (uri.getHost(), uri.getPort() == -1 ? 80 : uri.getPort());
    }
    catch (IOException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }
}

和TADA ......问题消失了。我会在Oracle上打开一个错误来表示问题。

答案 1 :(得分:0)

我运行了你的代码,它并没有像你说的那样停止。

没有太多时间来研究它为什么会这样,但我发现将执行程序服务的线程声明为守护程序会使问题消失:

private static ExecutorService TaskExecutor = Executors.newFixedThreadPool(5, new ThreadFactory() {
    public Thread newThread(Runnable r) {
        Thread t = new Thread(r);
        t.setDaemon(true);
        return t;
    }
});

如果我找到更好的解释,我会回来的。

答案 2 :(得分:0)

我认为当您尝试取消任务时,任务未启动的问题。我这样添加了Thread.sleep(100)

for (int i = 0; i < futures.length; ++i)
      futures[i] = startTask(taskExecutor);
Thread.sleep(100);
for (int i = 0; i < futures.length; ++i)
     System.out.println("futures[i].cancel(true): " + futures[i].cancel(true));

一切都被取消了。