Uploaded image for project: 'Hive'
  1. Hive
  2. HIVE-18050

LlapServiceDriver shoud split HIVE_AUX_JARS_PATH by ':' instead of ','

    XMLWordPrintableJSON

    Details

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

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Aegeaner Aegeaner
                Reporter:
                Aegeaner Aegeaner
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: