Служба виконавця Java зупинить, коли одне із призначених йому завдань не справляється з будь-якої причини


12

Мені потрібна якась послуга, яка виконуватиме декілька завдань одночасно та з інтервалом у 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. Впровадження Runnableповинні мати void runвизначення. Тип task1/2scheduleбуде ScheduledFuture<?>в наданому контексті. Переходячи до власне питання, як це можна використати awaitTermination? Ви могли це зробити так scheduledExecutorService.awaitTermination(1,TimeUnit.MINUTES);. Як варіант, що робити перевірка, чи скасоване будь-яке завдання перед його нормальним завершенням if (task1schedule.isCancelled() || task2schedule.isCancelled()) scheduledExecutorService.shutdown();:?
Наман

2
Немає сенсу планувати завдання, які потрібно повторювати щохвилини, але потім сказати, що ви хочете припинити завдання, "якби через одну хвилину все пішло добре". Оскільки ви зупиняєте виконавця в будь-якому випадку, планування завдання, яке закриває виконавця через одну хвилину, є тривіальним. А ф'ючерси вже вказують, чи пішло щось не так чи ні. Ви не сказали, який інший тип індикатора ви хочете.
Холгер

Відповіді:


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 годин - убийте завдання. це те, про що попросила ОП.
Грег Патнуде

Ви читали та розуміли мій коментар? Це не має значення , що робить цей код, він використовує збентежений клас без будь-якої причини, просто замінити TimerTaskз Runnableі ви виправили проблему, не змінюючи то , що робить цей код. Крім того, просто використовуйте, executor.schedule(assassin, ConfigurationOptions.getPollingCycleTime(), ChronoUnit.MINUTES);і він запуститься один раз у потрібний час, отже, if(LocalDateTime.now().isAfter(death))чек застарілий. Знову ж таки, це не змінює те, що робить код, крім того, що робить його значно простішим та ефективнішим.
Холгер
Використовуючи наш веб-сайт, ви визнаєте, що прочитали та зрозуміли наші Політику щодо файлів cookie та Політику конфіденційності.
Licensed under cc by-sa 3.0 with attribution required.