Mailing-List: contact issues-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Date: Sun, 14 Feb 2016 20:11:18 +0000 (UTC)
From: "Sean Busbey (JIRA)" <jira@apache.org>
To: issues@hbase.apache.org
Message-ID: <JIRA.12939038.1455386021000.50197.1455480678194@Atlassian.JIRA>
In-Reply-To: <JIRA.12939038.1455386021000@Atlassian.JIRA>
References: <JIRA.12939038.1455386021000@Atlassian.JIRA>
 <JIRA.12939038.1455386021205@arcas>
Subject: [jira] [Commented] (HBASE-15266) add precommit check for "catch
 Throwable"
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Sean Busbey commented on HBASE-15266:
-------------------------------------

Exceptions to a rule are likely to exist, and the judgement of our committers should be able to handle identifying appropriate uses and ensuring that we're handling things correctly (for example making sure we're not discarding Errors).

> add precommit check for "catch Throwable"
> -----------------------------------------
>
>                 Key: HBASE-15266
>                 URL: https://issues.apache.org/jira/browse/HBASE-15266
>             Project: HBase
>          Issue Type: New Feature
>          Components: test
>            Reporter: Sean Busbey
>            Priority: Minor
>
> Catching Throwable is usually incorrect because it gets all of the Error derived problems, like ThreadDeath, OutOfMemoryError, VM malfunctions, linker problems, etc.
> I was surprised to see findbugs did not flag catching throwable in the patches on HBASE-9393. We should add a precommit check that looks expressly for it, in our "hbase antipatterns" test plugin.


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)