hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Peter Vary (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HIVE-22060) Replacing "catch Throwable" with a more restricted exception class
Date Wed, 31 Jul 2019 12:45:00 GMT

    [ https://issues.apache.org/jira/browse/HIVE-22060?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16897132#comment-16897132
] 

Peter Vary commented on HIVE-22060:
-----------------------------------

[~isuller]: If done like this, I am sold on this idea :)

> Replacing "catch Throwable" with a more restricted exception class
> ------------------------------------------------------------------
>
>                 Key: HIVE-22060
>                 URL: https://issues.apache.org/jira/browse/HIVE-22060
>             Project: Hive
>          Issue Type: Improvement
>          Components: Hive
>            Reporter: Ivan Suller
>            Assignee: Ivan Suller
>            Priority: Major
>
> Catching Throwable considered unsafe in Java. A Throwable can be any Error and those
are JVM errors after the state of the JVM is not guaranteed thus the cleanest way to "handle"
the error is to let it kill the current thread.
> I ran a quick scan and found almost 400 "catch Throwable" in the current codebase. I
opened this ticket as a conversation starter to:
> - discuss if we want to eliminate this issue
> - if we want to do it what's the best way to do it



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

Mime
View raw message