8

苦痛なデバッグ経験の後、私はこの問題を追跡しました:ScheduledThreadPoolタスクが失敗した場合は報告せず、もう一度失敗したタスクを実行しません。したがって、定期的なジョブの活性を追跡することは困難であり、さらに他の定期的なタスク(デッドマンスイッチまたはを介してScheduledFuture)でそれらをチェックする必要があります。

これで、を渡すことができますがScheduledThreadPoolUncaughtExceptionHandlerそれでもうまくいかないようです。

import java.util.concurrent.*;

class Test {
  public static void main(String[] args) {
    final ThreadFactory tf = new ThreadFactory() {
      private final ThreadFactory delegate = Executors.defaultThreadFactory();

      @Override public Thread newThread(final Runnable r) {
        final Thread res = delegate.newThread(r);
        res.setUncaughtExceptionHandler(new Thread.UncaughtExceptionHandler() {
          @Override
          public void uncaughtException(final Thread t, final Throwable e) {
            e.printStackTrace();
          }
        });
        return res;
      }
    };
    final ScheduledThreadPoolExecutor exec = new ScheduledThreadPoolExecutor(1, tf);

    final Runnable task = new Runnable() {
      private int c = 0;

      @Override
      public void run() {
        if ( c++ == 5 ) {
          throw new ArrayIndexOutOfBoundsException("Runtime error!");
        }

        System.out.println("Reached " + c);
      }
    };

    exec.scheduleWithFixedDelay(task, 1, 1, TimeUnit.SECONDS);
  }
}

このプログラムの出力は単純です(Oracle Java SE(64ビットサーバー)1.7.0_06-b24)

Reached 1
Reached 2
Reached 3
Reached 4
Reached 5

そして、それは(設計上)ハングします。

私はいつでもタスク全体を試してみることができますが、それは醜い感じです。UncaughtExceptionHandlerすでにそれを行う必要があります!

この問題のAPIソリューションはありますか?私は何か間違ったことをしましたか、それともバグですか?

4

2 に答える 2

6

通貨スレッドプールはすべての例外をキャプチャし、Futureオブジェクトに配置して検査します。例外のためだけに、スレッドはスレッドをキャッチして強制終了UncaughtExceptionHandlerしません。この場合、スレッドプールコードによってスローされた例外に対してのみです。

これを回避する簡単な方法は、ランナブルをラップすることです。

public class ExceptionHandlingScheduledExecutor extends ScheduledThreadPoolExecutor {
    private final Thread.UncaughtExceptionHandler ueh;

    public ExceptionHandlingScheduledExecutor(int corePoolSize, Thread.UncaughtExceptionHandler ueh) {
        super(corePoolSize);
        this.ueh = ueh;
    }

    @Override
    public ScheduledFuture<?> schedule(Runnable command, long delay, TimeUnit unit) {
        return super.schedule(wrap(command), delay, unit);
    }

    @Override
    public <V> ScheduledFuture<V> schedule(Callable<V> callable, long delay, TimeUnit unit) {
        return super.schedule(wrap(callable), delay, unit); 
    }

    @Override
    public ScheduledFuture<?> scheduleAtFixedRate(Runnable command, long initialDelay, long period, TimeUnit unit) {
        return super.scheduleAtFixedRate(wrap(command), initialDelay, period, unit);
    }

    @Override
    public ScheduledFuture<?> scheduleWithFixedDelay(Runnable command, long initialDelay, long delay, TimeUnit unit) {
        return super.scheduleWithFixedDelay(wrap(command), initialDelay, delay, unit);
    }

    @Override
    public void execute(Runnable command) {
        super.execute(wrap(command));
    }

    @Override
    public Future<?> submit(Runnable task) {
        return super.submit(wrap(task));
    }

    @Override
    public <T> Future<T> submit(Runnable task, T result) {
        return super.submit(wrap(task), result);
    }

    @Override
    public <T> Future<T> submit(Callable<T> task) {
        return super.submit(wrap(task));
    }

    private Runnable wrap(final Runnable runnable) {
        return new Runnable() {
            @Override
            public void run() {
                try {
                    runnable.run();
                } catch (final Throwable t) {
                    ueh.uncaughtException(Thread.currentThread(), t);
                    throw t;
                }
            }
        };
    }

    private <T> Callable<T> wrap(final Callable<T> callable) {
        return new Callable<T>() {
            @Override
            public T call() throws Exception {
                try {
                    return callable.call();
                } catch (Throwable t) {
                    ueh.uncaughtException(Thread.currentThread(), t);
                    throw t;
                }
            }
        };
    }
}

ThreadPoolExecutorをサブクラス化して、これを透過的に行うことができます。


キャッシュされたスレッドプールを使用して例外を処理することもできますが、これはより複雑です。

返さFutureれたものを透過的に使用する1つの方法は、サブクラスScheduledThreadPoolExecutor(または、さらに言えば、任意のエグゼキューター)を使用することです。

class MyScheduledExecutor extends ScheduledThreadPoolExecutor {
  private final Thread.UncaughtExceptionHandler ueh;
  private final ExecutorService futureService = Executors.newCachedThreadPool();

  public MyScheduledExecutor(int corePoolSize, Thread.UncaughtExceptionHandler ueh) {
    super(corePoolSize);
    this.ueh = ueh;
  }

  // Copy other constructors

  @Override
  public ScheduledFuture<?> scheduleWithFixedDelay(Runnable command,
                                                   long initialDelay,
                                                   long delay,
                                                   TimeUnit unit) {
    final ScheduledFuture<?> f = super.scheduleWithFixedDelay(command, initialDelay, delay, unit);
    futureService.submit(new Runnable() {
      @Override
      public void run() {
        try {
          f.get();
        } catch (Throwable t ) {
          ueh.uncaughtException(null, t.getCause());
        }
      }
    };

    return f;
  }

  // Do similarly for other submit/schedule methods
}

そして、次のように使用します。

final ScheduledThreadPoolExecutor exec = new MyScheduledExecutor(1, new Thread.UncaughtExceptionHandler() {
      @Override
      public void uncaughtException(final Thread t, final Throwable e) {
        e.printStackTrace();
      }
    });

これで、出力は希望どおりになります。

Reached 1
Reached 2
Reached 3
Reached 4
Reached 5
java.lang.ArrayIndexOutOfBoundsException: Runtime error!
   ...
于 2012-08-29T10:52:01.597 に答える
2

上記で提案されたラッピングを行うjcabi-logのVerboseRunnableクラスを使用できます。

import com.jcabi.log.VerboseRunnable;
Runnable runnable = new VerboseRunnable(
  Runnable() {
    public void run() { 
      // do business logic, may Exception occurs
    }
  },
  true // it means that all exceptions will be swallowed and logged
);

これで、エグゼキュータが呼び出しrunnable.run()ても例外はスローされません。代わりに、それらは飲み込まれ、ログに記録されます(SLF4Jに)。したがって、エグゼキュータは例外のために停止せず、何が起こっているかがわかります。

于 2013-04-06T05:40:43.173 に答える