Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Unhandled exceptions with Java scheduled executors

I have the following issue and I would like to know what exactly happens. I am using Java's ScheduledExecutorService to run a task every five minutes. It works very well. Executors completely changed the way I do thread programming in Java.

Now, I browsed Java Doc for information about what would be the behavior in case the scheduled task fails with an unhandled exception, but couldn't find anything.

Is the next scheduled task still going to run? If there is an unhandled exception, the scheduled executor stops scheduling task? Can anyone point to information regarding this simple issue?

Thanks a lot.

like image 958
Pablo Santa Cruz Avatar asked Nov 02 '09 11:11

Pablo Santa Cruz


People also ask

How do you catch an exception inside a scheduler function?

Simplest Approach: Just Add a Try-Catch Always wrap your Runnable's code in a Try-Catch to catch any and all exceptions and errors.


3 Answers

The Javadoc of both scheduleAtFixedRate and scheduleWithFixedDelay says "If any execution of the task encounters an exception, subsequent executions are suppressed." I don't find that to be exactly crystal clear, but it seems to be saying that if your run method throws any kind of exception, then the scheduler will effectively drop that task. Any other tasks running via that scheduler should not be affected. It shouldn't be hard to test what it actually does...

Cancellation of the task may not necessarily be a bad thing. If the run method throws a RuntimeException, it's probably got a bug somewhere, and the state of the system is unknown. But at minimum I would advise catching RuntimeException in your run method, and logging the full stack trace at SEVERE. You may want to then rethrow to cancel the task, depending on the circumstances. But either way you'll need the logging to have a fighting chance of working out what went wrong.

like image 171
Lachlan Avatar answered Oct 19 '22 14:10

Lachlan


If you are using scheduleAtFixedRate() or scheduleAtFixedDelay(), and your task bails out with an exception, that task will not be rescheduled. However, other independent tasks should continue to execute as expected. (See API Docs). If you care that this has happened, you can grab ahold of the ScheduledFuture that is returned and call the get() method. If the underlying task tosses an exception, you'll get it thrown out of the get() method, wrapped in an ExecutionException.

like image 39
andersoj Avatar answered Oct 19 '22 14:10

andersoj


This man had the same problem.

http://code.nomad-labs.com/2011/12/09/mother-fk-the-scheduledexecutorservice/

His solution is to catch Exception inside the runnable and re throw an RuntimeException:

try {
       theRunnable.run();
    } catch (Exception e) {
       // LOG IT HERE!!!
       System.err.println("error in executing: " + theRunnable + ". It will no longer be run!");
       e.printStackTrace();

       // and re throw it so that the Executor also gets this error so that it can do what it would
       // usually do
       throw new RuntimeException(e);
}
like image 3
mrclrchtr Avatar answered Oct 19 '22 16:10

mrclrchtr