割り当てられたタスクの1つが何らかの理由で失敗すると、Javaはexecutorサービスを停止します


12

いくつかのタスクを同時に、1秒の間隔で1分間実行するある種のサービスが必要です。

タスクの1つが失敗した場合、サービスとそれとともに実行されたすべてのタスクを何らかの問題が発生したことを示すインジケーターで停止します。それ以外の場合は、1分後にすべてがうまくいった場合、サービスはすべて正常に終了したことを示すインジケーターで停止します。

たとえば、私は2つの機能があります:

Runnable task1 = ()->{
      int num = Math.rand(1,100);
      if (num < 5){
          throw new Exception("something went wrong with this task,terminate");
      }
}

Runnable task2 = ()->{
      int num = Math.rand(1,100)
      return num < 50;
}



ScheduledExecutorService scheduledExecutorService = Executors.newScheduledThreadPool(2);
task1schedule = scheduledExecutorService.scheduleAtFixedRate(task1, 1, 60, TimeUnit.SECONDS);
task2schedule = scheduledExecutorService.scheduleAtFixedRate(task2, 1, 60, TimeUnit.SECONDS);

if (!task1schedule || !task2schedule) scheduledExecutorService.shutdown();

これにどのように取り組み、可能な限り一般的なものにするかについてのアイデアはありますか?


1
実際の質問とMath.randは別に、組み込みAPI ではありません。の実装にRunnablevoid run定義が必要です。タイプはtask1/2scheduleなりScheduledFuture<?>提供されるコンテキストで。実際の質問に移りますが、それをどのように活用するのawaitTerminationですか?あなたはそれをとして行うことができますscheduledExecutorService.awaitTermination(1,TimeUnit.MINUTES);。または、通常の完了前にキャンセルされたタスクがあるかどうかを確認するのはどうif (task1schedule.isCancelled() || task2schedule.isCancelled()) scheduledExecutorService.shutdown();ですか?
ナマン

2
タスクを毎分繰り返すようにスケジュールするのは意味がありませんが、「1分後にすべてがうまくいった場合は」タスクを停止したいとします。どちらの場合もexecutorを停止しているため、1分後にexecutorをシャットダウンするタスクをスケジュールするのは簡単です。そして、先物は何かがうまくいかなかったかをすでに示しています。他にどのような種類のインジケーターが必要かは言いませんでした。
ホルガー

回答:


8

アイデアは、タスクが共通オブジェクトTaskCompleteEventにプッシュしているというものです。彼らがエラーをプッシュすると、スケジューラーは停止し、すべてのタスクが停止します。

すべてのタスク反復の結果は、マップ「エラー」と「成功」で確認できます。

public class SchedulerTest {

    @Test
    public void scheduler() throws InterruptedException {
        ScheduledExecutorService scheduledExecutorService = Executors.newScheduledThreadPool(2);
        TaskCompleteEvent taskCompleteEvent = new TaskCompleteEvent(scheduledExecutorService);
        Runnable task1 = () -> {
            int num = new Random().nextInt(100);
            if (num < 5) {
                taskCompleteEvent.message("task1-"+UUID.randomUUID().toString(), "Num "+num+" was obatined. Breaking all the executions.", true);
            }
        };
        Runnable task2 = () -> {
            int num = new Random().nextInt(100);
            taskCompleteEvent.message("task2-"+UUID.randomUUID().toString(), num < 50, false);
        };
        scheduledExecutorService.scheduleAtFixedRate(task1, 0, 1, TimeUnit.SECONDS);
        scheduledExecutorService.scheduleAtFixedRate(task2, 0, 1, TimeUnit.SECONDS);
        scheduledExecutorService.awaitTermination(60, TimeUnit.SECONDS);
        System.out.println("Success: "+taskCompleteEvent.getSuccess());
        System.out.println("Errors: "+taskCompleteEvent.getErrors());
        System.out.println("Went well?: "+taskCompleteEvent.getErrors().isEmpty());
    }

    public static class TaskCompleteEvent {

        private final ScheduledExecutorService scheduledExecutorService;
        private final Map<String, Object> errors = new LinkedHashMap<>();
        private final Map<String, Object> success = new LinkedHashMap<>();

        public TaskCompleteEvent(ScheduledExecutorService scheduledExecutorService) {
            this.scheduledExecutorService = scheduledExecutorService;
        }

        public synchronized void message(String id, Object response, boolean error) {
            if (error) {
                errors.put(id, response);
                scheduledExecutorService.shutdown();
            } else {
                success.put(id, response);
            }
        }

        public synchronized Map<String, Object> getErrors() {
            return errors;
        }

        public synchronized Map<String, Object> getSuccess() {
            return success;
        }

    }

}

2

他のすべての実行中のタスクを監視するジョブを持つ追加のタスクを追加する必要があるだけです。監視対象のタスクのいずれかが失敗した場合、アサシンが検査できるセマフォ(フラグ)を設定する必要があります。

    ScheduledExecutorService executor = (ScheduledExecutorService) Executors.newScheduledThreadPool(2);

    // INSTANTIATE THE REMOTE-FILE-MONITOR:
    RemoteFileMonitor monitor = new RemoteFileMonitor(remotesource, localtarget);

    // THIS TimerTask PERIODICALLY TRIGGERS THE RemoteFileMonitor: 
    TimerTask remote = new TimerTask() {

        // RUN FORREST... RUN !
        public void run() {

            try { 

                kae.trace("TimerTask::run() --> Calling RemoteFileMonitor.check()");
                monitor.check();

            } catch (Exception ex) {

                // NULL TRAP: ALLOWS US TO CONTINUE AND RETRY:

            }

        }

    };

    // THIS TimerTask PERIODICALLY TRIES TO KILL THE REMOTE-FILE-MONITOR:
    TimerTask assassin = new TimerTask() {

        // WHERE DO BAD FOLKS GO WHEN THEY DIE ? 
        private final LocalDateTime death = LocalDateTime.now().plus(ConfigurationOptions.getPollingCycleTime(), ChronoUnit.MINUTES);

        // RUN FORREST... RUN !
        public void run() {

            // IS THERE LIFE AFTER DEATH ???
            if (LocalDateTime.now().isAfter(death)) {

                // THEY GO TO A LAKE OF FIRE AND FRY:
                kae.error(ReturnCode.MONITOR_POLLING_CYCLE_EXCEEDED);                   

            }

        }

    };

    // SCHEDULE THE PERIODIC EXECUTION OF THE RemoteFileMonitor: (remote --> run() monitor --> check())
    executor.scheduleAtFixedRate(remote, delay, interval, TimeUnit.MINUTES);

    // SCHEDULE PERIODIC ASSASSINATION ATTEMPTS AGAINST THE RemoteFileMonitor: (assassin --> run() --> after death --> die())
    executor.scheduleAtFixedRate(assassin, delay, 60L, TimeUnit.SECONDS);

    // LOOP UNTIL THE MONITOR COMPLETES:
    do {

        try {

            // I THINK I NEED A NAP:
            Thread.sleep(interval * 10);                

        } catch (InterruptedException e) {

            // FAIL && THEN cleanexit();
            kae.error(ReturnCode.MONITORING_ERROR, "Monitoring of the XXXXXX-Ingestion site was interrupted");

        }

        // NOTE: THE MONITOR IS SET TO 'FINISHED' WHEN THE DONE-File IS DELIVERED AND RETRIEVED:
    } while (monitor.isNotFinished());

    // SHUTDOWN THE MONITOR TASK:
    executor.shutdown();

2
クラスTimerTaskはとはまったく無関係ScheduledExecutorServiceです。たまたま実装するだけRunnableです。さらに、特定の時間(ConfigurationOptions.getPollingCycleTime())に達したかどうかを確認するだけで、定期的なタスクをスケジュールすることは意味がありません。あなたはを持っているScheduledExecutorServiceので、希望する時間にタスクを正しくスケジュールするように指示できます。
ホルガー

私が使用した例の実装は、タスクが完了していない場合に、一定期間後に実行中のタスクを強制終了することでした。使用例は次のとおりです。リモートサーバーが2時間以内にファイルをドロップしなかった場合-タスクを強制終了します。これはOPが要求したものです。
Greg Patnude、

私のコメントを読んで理解しましたか?それはコードが何をするかは関係ありません。理由もなく落胆したクラスを使用し、コードを何も変更せずに置き換えTimerTaskRunnable、問題を修正しました。さらに、使用executor.schedule(assassin, ConfigurationOptions.getPollingCycleTime(), ChronoUnit.MINUTES);するだけで、必要なときに1回実行されるため、if(LocalDateTime.now().isAfter(death))チェックは廃止されます。繰り返しますが、はコードの動作を変更しませんが、大幅にシンプルで効率的になります。
ホルガー
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.