Either re-interrupt this method or rethrow the "InterruptedException issue in sonar

Question

In one of my method , interrupted exception and execution exception is coming. I put in try catch like this.

try{

  //my code
}catch(InterruptedException|ExecutionException e)

  Log.error(" logging it");
  throw new MonitoringException("it failed" , e)


//monitoringexception extends RunTimeException

Also in my method I put throws InterruptedException,ExecutionException

I am getting below critical error in sonar - Either re-interrupt this method or rethrow the "InterruptedException"

Anyone know how to fix this.

Please help immediately.

See https://stackoverflow.com/questions/3976344/handling-interruptedexception-in-java — Ori Marko, Jan 30 '19 at 07:10

jumping_monkey · Answer 1 · 2020-04-23T04:02:21.403

To "re-interrupt" as a best practice:

try{
    //some code
} catch (InterruptedException ie) {
    logger.error("InterruptedException: ", ie);
    Thread.currentThread().interrupt();
} catch (ExecutionException ee) {
    logger.error("ExecutionException: ",ee);
}

Usually, when a thread is interrupted, whoever is interrupting the thread, wants the thread to exit what it's currently doing.

However, make sure that you do NOT multi-catch:

catch (InterruptedException | ExecutionException e) {     
  logger.error("An error has occurred: ", e);
  Thread.currentThread().interrupt();
}

We do not want ExecutionException to be "re-interrupted".

BONUS:
If you are interested, you can play with the examples here

Cheers

Weird my vscode says that the second catch is unrechable code. — Jawad El Fou, Oct 10 '21 at 01:13

Either re-interrupt this method or rethrow the "InterruptedException issue in sonar

1 Answers1