hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Heng Chen (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-15266) add precommit check for "catch Throwable"
Date Sun, 14 Feb 2016 09:18:18 GMT

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

Heng Chen commented on HBASE-15266:

IMO some monitor threads need to catch throwable.

> 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

This message was sent by Atlassian JIRA

View raw message