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

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

Log workAgile BoardRank to TopRank to BottomVotersWatch issueWatchersCreate sub-taskConvert to sub-taskMoveLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    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

          Activity

          $i18n.getText('security.level.explanation', $currentSelection) Viewable by All Users
          Cancel

            People

              Dates

              • Created:
                Updated:
                Resolved:

                Issue deployment