LoggingResearch opened a new pull request, #3554:
URL: https://github.com/apache/storm/pull/3554

   ## What is the purpose of the change
   
   This PR addresses the handling of InterruptedException in the close() method 
of the Supervisor class. The changes include:
   
   Adding a separate catch block for InterruptedException that is thrown by the 
close() method of the AsyncLocalizer class.
   Logging the InterruptedException separately to ensure that the exception 
type and message are correctly recorded in the logs.
   The motivation behind this change is to properly handle the scenario where 
the close() method of AsyncLocalizer is interrupted, which is a situation that 
was not specifically addressed in the original code. By catching and logging 
the InterruptedException separately, we can provide more specific information 
in the logs when such an event occurs.
   
   ## How was the change tested
   
   Built storm-server. Validated logging on dev supervisor.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@storm.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to