hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sushanth Sowmyan (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HIVE-10267) HIVE-9664 makes hive depend on ivysettings.xml : trivial breakage fix
Date Fri, 10 Apr 2015 03:48:12 GMT

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

Sushanth Sowmyan commented on HIVE-10267:
-----------------------------------------

>From http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build/3351/testReport/
, there were no regressions, and all the noted failures here do not seem connected to this
patch. 

> HIVE-9664 makes hive depend on ivysettings.xml : trivial breakage fix
> ---------------------------------------------------------------------
>
>                 Key: HIVE-10267
>                 URL: https://issues.apache.org/jira/browse/HIVE-10267
>             Project: Hive
>          Issue Type: Bug
>    Affects Versions: 1.2.0
>            Reporter: Sushanth Sowmyan
>            Assignee: Sushanth Sowmyan
>              Labels: patch
>         Attachments: HIVE-10267.patch
>
>
> HIVE-9664 makes hive depend on the existence of ivysettings.xml, and if it is not present,
it makes hive NPE when instantiating a CLISessionState.
> {noformat}
> java.lang.NullPointerException
> at org.apache.hadoop.hive.ql.session.DependencyResolver.<init>(DependencyResolver.java:61)
> at org.apache.hadoop.hive.ql.session.SessionState.<init>(SessionState.java:343)
> at org.apache.hadoop.hive.ql.session.SessionState.<init>(SessionState.java:334)
> at org.apache.hadoop.hive.cli.CliSessionState.<init>(CliSessionState.java:60)
> {noformat}
> This happens because of the following bit:
> {noformat}
>     // If HIVE_HOME is not defined or file is not found in HIVE_HOME/conf then load default
ivysettings.xml from class loader
>     if (ivysettingsPath == null || !(new File(ivysettingsPath).exists())) {
>       ivysettingsPath = ClassLoader.getSystemResource("ivysettings.xml").getFile();
>       _console.printInfo("ivysettings.xml file not found in HIVE_HOME or HIVE_CONF_DIR,"
+ ivysettingsPath + " will be used");
>     }
> {noformat}
> This makes it so that an attempt to instantiate CliSessionState without an ivysettings.xml
file will cause hive to fail with an NPE. Hive should not have a hard dependency on a ivysettings,xml
being present, and this feature should gracefully fail in that case instead.



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

Mime
View raw message