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

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

              Dates

                Created:
                Updated:
                Resolved: