XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • HDFS-7240
    • hdfs
    • None
    • Reviewed

    Description

      Current codebase does:

            hadoop_debug "Appending HADOOP_CBLOCK_OPTS onto HADOOP_OPTS" 
            HADOOP_OPTS="${HADOOP_OPTS} ${HADOOP_CBLOCK_OPTS}"
      
      ...  
      
            hadoop_debug "Appending HADOOP_JSCSI_OPTS onto HADOOP_OPTS"
            HADOOP_OPTS="${HADOOP_OPTS} ${HADOOP_JSCSI_OPTS}"
      

      This code block breaks the consistency with the rest of the shell scripts:

      a) It should be HDFS_CBLOCKSERVER_OPTS, HDFS_JSCSI_OPTS
      b) _OPTS, unless they are a deprecated form or some other special case, are already/automatically appended; there is no need to do them specifically

      Also, a description of these env vars should be hadoop-env.sh for documentation purposes.

      Attachments

        1. HDFS-11972-HDFS-7240.001.patch
          4 kB
          Mukul Kumar Singh

        Activity

          People

            msingh Mukul Kumar Singh
            aw Allen Wittenauer
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: