hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HIVE-3799) Better error message if metalisteners or hookContext cannot be loaded/instantiated
Date Mon, 28 Jan 2013 10:29:14 GMT

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

Hudson commented on HIVE-3799:
------------------------------

Integrated in Hive-trunk-h0.21 #1941 (See [https://builds.apache.org/job/Hive-trunk-h0.21/1941/])
    HIVE-3799 : Better error message if metalisteners or hookContext cannot be loaded/instantiated
(Navis via Ashutosh Chauhan) (Revision 1439138)

     Result = FAILURE
hashutosh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439138
Files : 
* /hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java
* /hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestMetaStoreListenersError.java

                
> Better error message if metalisteners or hookContext cannot be loaded/instantiated
> ----------------------------------------------------------------------------------
>
>                 Key: HIVE-3799
>                 URL: https://issues.apache.org/jira/browse/HIVE-3799
>             Project: Hive
>          Issue Type: Improvement
>          Components: Diagnosability, Logging
>            Reporter: Sudhanshu Arora
>            Assignee: Navis
>            Priority: Trivial
>             Fix For: 0.11.0
>
>         Attachments: HIVE-3799.D7719.1.patch, HIVE-3799.D7719.2.patch
>
>
> If I am not able to instantiate MetaListener or HookContext because of some dependent
class missing, the only error I get is InvocationTargetException.  MetaStoreUtils.getMetaListener
should be changed as following:
> try {
>         T listener = (T) Class.forName(
>             listenerImpl.trim(), true, JavaUtils.getClassLoader()).getConstructor(
>                 Configuration.class).newInstance(conf);
>         listeners.add(listener);
>       } catch (InvocationTargetException e) {
>         throw new MetaException("Failed to instantiate listener named: "+
>             listenerImpl + " reason: " + e.getCause().toString());
>       } catch (Exception e) {
>         throw new MetaException("Failed to instantiate listener named: "+
>             listenerImpl + " reason: " + e.toString());
>       }
> Similarly Driver.getHooks should be changed to handle InvocationTargetException in a
better way.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message