You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@storm.apache.org by "Richard Zowalla (Jira)" <ji...@apache.org> on 2023/11/09 12:52:00 UTC

[jira] [Closed] (STORM-3938) Unhandled InterruptedException in Supervisor's close() method

     [ https://issues.apache.org/jira/browse/STORM-3938?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Richard Zowalla closed STORM-3938.
----------------------------------
      Assignee: Richard Zowalla
    Resolution: Won't Fix

I checked out the PR / branch and the IDE reports, that the given exception isn't thrown there resulting in compile errors. Feel free to re-open this ticket again, if you can provide a more concise explanation of the actual problem.

> Unhandled InterruptedException in Supervisor's close() method
> -------------------------------------------------------------
>
>                 Key: STORM-3938
>                 URL: https://issues.apache.org/jira/browse/STORM-3938
>             Project: Apache Storm
>          Issue Type: Improvement
>          Components: storm-server
>    Affects Versions: 2.4.0
>            Reporter: LoggingResearch
>            Assignee: Richard Zowalla
>            Priority: Blocker
>          Time Spent: 1h
>  Remaining Estimate: 0h
>
> In the Supervisor class's close() method, we are attempting to close the asyncLocalizer object which can throw an InterruptedException. However, currently, the InterruptedException thrown by asyncLocalizer.close() is not being caught or logged. This is an issue because InterruptedException is a more specific exception that might need to be handled separately from the general Exception catch block.
> This can lead to a situation where if an InterruptedException is thrown, we will not have any log information about it, making it difficult to debug the issue.
> Proposed solution:
> Add a separate catch block for InterruptedException before the general Exception catch block to handle and log the InterruptedException properly.
> https://github.com/apache/storm/pull/3554



--
This message was sent by Atlassian Jira
(v8.20.10#820010)