You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hive.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/11/13 10:52:00 UTC
[jira] [Updated] (HIVE-18050) LlapServiceDriver shoud split
HIVE_AUX_JARS_PATH by ':' instead of ','
[ https://issues.apache.org/jira/browse/HIVE-18050?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
ASF GitHub Bot updated HIVE-18050:
----------------------------------
Labels: pull-request-available (was: )
> LlapServiceDriver shoud split HIVE_AUX_JARS_PATH by ':' instead of ','
> ----------------------------------------------------------------------
>
> Key: HIVE-18050
> URL: https://issues.apache.org/jira/browse/HIVE-18050
> Project: Hive
> Issue Type: Bug
> Components: CLI, Clients
> Affects Versions: 2.3.0
> Reporter: Aegeaner
> Assignee: Aegeaner
> Labels: pull-request-available
>
> LlapServiceDriver shoud split HIVE_AUX_JARS_PATH by ':' instead of ',' , since in hive script the environment variable has been replaced:
> {code:java}
> elif [ "${HIVE_AUX_JARS_PATH}" != "" ]; then
> HIVE_AUX_JARS_PATH=`echo $HIVE_AUX_JARS_PATH | sed 's/,/:/g'`
> if $cygwin; then
> HIVE_AUX_JARS_PATH=`cygpath -p -w "$HIVE_AUX_JARS_PATH"`
> HIVE_AUX_JARS_PATH=`echo $HIVE_AUX_JARS_PATH | sed 's/;/,/g'`
> fi
> AUX_CLASSPATH=${AUX_CLASSPATH}:${HIVE_AUX_JARS_PATH}
> AUX_PARAM="file://$(echo ${HIVE_AUX_JARS_PATH} | sed 's/:/,file:\/\//g')"
> fi
> {code}
> But in the LLAP Service Driver, it's processed as :
> {code:java}
> private void addAuxJarsToSet(HashSet<String> auxJarSet, String auxJars) {
> if (auxJars != null && !auxJars.isEmpty()) {
> // TODO: transitive dependencies warning?
> String[] jarPaths = auxJars.split(",");
> for (String jarPath : jarPaths) {
> if (!jarPath.isEmpty()) {
> auxJarSet.add(jarPath);
> }
> }
> }
> }
> };
> {code}
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)