[HIVE-18050] LlapServiceDriver shoud split HIVE_AUX_JARS_PATH by ':' instead of ','

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.3.0
• Fix Version/s: 3.0.0
• Component/s: CLI, Clients
• Labels: pull-request-available
• Assignee: Aegeaner
• Reporter: Aegeaner
• Created: 13/Nov/17 09:17
• Updated: 22/May/18 23:13
• Resolved: 21/Nov/17 22:01

Description

LlapServiceDriver shoud split HIVE_AUX_JARS_PATH by ':' instead of ',' , since in hive script the environment variable has been replaced:

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

But in the LLAP Service Driver, it's processed as :

 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);
              }
            }
          }
        }
      };

Issue Links

• links to: GitHub Pull Request #271