[jira] [Created] (HIVE-18050) LlapServiceDriver shoud split HIVE_AUX_JARS_PATH by ':' instead of ','

[Aegeaner (JIRA)]

Aegeaner created HIVE-18050:
-------------------------------

             Summary: 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


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)