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

Zhihong Ted Yu commented on HBASE-6639:
---------------------------------------

Looks like there is more than one place where Exception is not caught:
{code}
      o = implClass.newInstance();
./hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/CoprocessorHost.java
      HLog.Reader reader = logReaderClass.newInstance();
      HLog.Writer writer = (HLog.Writer) logWriterClass.newInstance();
      return keyClass.newInstance();
./hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLog.java
          key = keyClass.newInstance();
./hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/SequenceFileLogReader.java
{code}
                
> Class.newInstance() can throw any checked exceptions and must be encapsulated 
> with catching Exception
> -----------------------------------------------------------------------------------------------------
>
>                 Key: HBASE-6639
>                 URL: https://issues.apache.org/jira/browse/HBASE-6639
>             Project: HBase
>          Issue Type: Bug
>    Affects Versions: 0.94.1
>            Reporter: Hiroshi Ikeda
>            Priority: Minor
>
> There are some logics to call Class.newInstance() without catching Exception,
> for example, in the method CoprocessorHost.loadInstance().
> Class.newInstance() is declared to throw InstantiationException and 
> IllegalAccessException but indeed the method can throw any checked exceptions 
> without declaration.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to