[
https://issues.apache.org/jira/browse/HIVE-18050?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16250825#comment-16250825
]
Aegeaner commented on HIVE-18050:
---------------------------------
That's right, should call addAuxJarsToSet with different delimiters.
> 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)
