Java interrompe il servizio di esecuzione quando una delle attività assegnate non riesce per qualsiasi motivo


12

Ho bisogno di un tipo di servizio che eseguirà alcune attività contemporaneamente e in un intervallo di 1 secondo per 1 minuto.

Se una delle attività fallisce, desidero interrompere il servizio e tutte le attività che lo hanno eseguito con una sorta di indicatore che qualcosa è andato storto, altrimenti se dopo un minuto tutto è andato bene il servizio si fermerà con un indicatore che tutto è andato bene.

Ad esempio, ho 2 funzioni:

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();

Qualche idea su come dovrei affrontarlo e rendere le cose il più generiche possibile?


1
Poche cose a parte la vera domanda, Math.randnon è un'API integrata. Un'implementazione di Runnabledeve avere una void rundefinizione. Il tipo di task1/2schedulesarebbe ScheduledFuture<?>nel contesto fornito. Passando alla domanda vera e propria, come può essere utilizzata awaitTermination? Potresti farlo come scheduledExecutorService.awaitTermination(1,TimeUnit.MINUTES);. In alternativa, che ne dici di verificare se una delle attività è stata annullata prima del suo normale completamento if (task1schedule.isCancelled() || task2schedule.isCancelled()) scheduledExecutorService.shutdown();:?
Naman,

2
Non ha senso ripetere le attività di pianificazione ogni minuto, ma poi dire, si desidera interrompere le attività "se dopo un minuto tutto è andato bene". Dato che in entrambi i casi stai fermando l'esecutore, programmare un'attività che spegne l'esecutore dopo un minuto è banale. E i futuri indicano già se qualcosa è andato storto o no. Non hai detto quale altro tipo di indicatore vuoi.
Holger,

Risposte:


8

L'idea è che le attività vengano inviate a un oggetto comune TaskCompleteEvent. Se si verifica un errore, lo scheduler viene arrestato e tutte le attività verranno interrotte.

Puoi controllare i risultati di ogni iterazione delle attività nelle mappe "errori" e "esito positivo".

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

Devi solo aggiungere un'attività aggiuntiva il cui compito è monitorare tutte le altre attività in esecuzione - e quando una qualsiasi delle attività monitorate fallisce, devono impostare un semaforo (flag) che l'assassino può ispezionare.

    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
La classe non TimerTaskè completamente estranea a ScheduledExecutorService; capita solo di implementare Runnable. Inoltre, non ha senso pianificare un'attività periodica, solo per verificare se ConfigurationOptions.getPollingCycleTime()è stato raggiunto un determinato orario ( ). Hai un ScheduledExecutorService, quindi puoi dirlo per pianificare l'attività nel momento desiderato.
Holger,

L'implementazione nell'esempio che ho usato è stata quella di terminare un'attività in esecuzione dopo un certo periodo di tempo se l'attività non era stata completata. Il caso d'uso era: se il server remoto non ha lasciato cadere un file entro 2 ore, interrompere l'attività. questo è ciò che l'OP ha richiesto.
Greg Patnude,

Hai letto e compreso il mio commento? Non importa ciò che fa il codice, utilizza una classe scoraggiata senza motivo, basta sostituirlo TimerTaskcon Runnablee il problema è stato risolto, senza modificare ciò che fa il codice. Inoltre, basta usare executor.schedule(assassin, ConfigurationOptions.getPollingCycleTime(), ChronoUnit.MINUTES);e verrà eseguito una volta all'ora desiderata, quindi il if(LocalDateTime.now().isAfter(death))controllo è obsoleto. Ancora una volta, il non cambia ciò che fa il codice, oltre a farlo sostanzialmente più semplice ed efficiente.
Holger,
Utilizzando il nostro sito, riconosci di aver letto e compreso le nostre Informativa sui cookie e Informativa sulla privacy.
Licensed under cc by-sa 3.0 with attribution required.