Hadoop Common
  1. Hadoop Common
  2. HADOOP-7491

hadoop command should respect HADOOP_OPTS when given a class name

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.23.0
    • Fix Version/s: 0.23.0
    • Component/s: scripts
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      When using the hadoop command HADOOP_OPTS and HADOOP_CLIENT_OPTS options are not passeed through.

        Issue Links

          Activity

          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Patch Available Patch Available
          44m 33s 1 Aaron T. Myers 29/Jul/11 22:38
          Patch Available Patch Available Resolved Resolved
          39m 47s 1 Eli Collins 29/Jul/11 23:18
          Resolved Resolved Closed Closed
          108d 2h 31m 1 Arun C Murthy 15/Nov/11 00:50
          Arun C Murthy made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Eli Collins made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Eli Collins added a comment -

          I've committed this. Thanks atm.

          Show
          Eli Collins added a comment - I've committed this. Thanks atm.
          Eli Collins made changes -
          Hadoop Flags [Reviewed]
          Summary hadoop command doesn't use HADOOP_*_OPTS when given a class name hadoop command should respect HADOOP_OPTS when given a class name
          Hide
          Eli Collins added a comment -

          Thanks atm. I concur. Btw verified by hand that HADOOP_OPTS is now respected since Hudson won't pick up on this.

          common $ export HADOOP_OPTS="-INVALID"
          common $ ./bin/hadoop org.apache.hadoop.hdfs.tools.HDFSConcat
          Unrecognized option: -INVALID
          Could not create the Java virtual machine.
          
          Show
          Eli Collins added a comment - Thanks atm. I concur. Btw verified by hand that HADOOP_OPTS is now respected since Hudson won't pick up on this. common $ export HADOOP_OPTS="-INVALID" common $ ./bin/hadoop org.apache.hadoop.hdfs.tools.HDFSConcat Unrecognized option: -INVALID Could not create the Java virtual machine.
          Hide
          Aaron T. Myers added a comment -

          +1, patch looks good to me.

          No idea why we didn't do this in the first place. Probably just an oversight.

          Show
          Aaron T. Myers added a comment - +1, patch looks good to me. No idea why we didn't do this in the first place. Probably just an oversight.
          Aaron T. Myers made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Affects Version/s 0.23.0 [ 12315569 ]
          Aaron T. Myers made changes -
          Assignee Eli Collins [ eli ]
          Eli Collins made changes -
          Attachment hadoop-7491-1.patch [ 12488245 ]
          Hide
          Eli Collins added a comment -

          Patch attached.

          Anyone know why we didn't do this from the start? I can't think of a reason why you wouldn't want to respect HADOOP_OPTS and HADOOP_CLIENT_OPTS when using bin/hadoop to run a class.

          Show
          Eli Collins added a comment - Patch attached. Anyone know why we didn't do this from the start? I can't think of a reason why you wouldn't want to respect HADOOP_OPTS and HADOOP_CLIENT_OPTS when using bin/hadoop to run a class.
          Hide
          Eli Collins added a comment -

          Before HADOOP-7325 users could specify options on the command line before the class name (instead of using HADOOP_OPTS), but can no longer do so because class name that start with "-" are no longer accepted.

          Show
          Eli Collins added a comment - Before HADOOP-7325 users could specify options on the command line before the class name (instead of using HADOOP_OPTS), but can no longer do so because class name that start with "-" are no longer accepted.
          Eli Collins made changes -
          Field Original Value New Value
          Link This issue is part of HADOOP-7325 [ HADOOP-7325 ]
          Eli Collins created issue -

            People

            • Assignee:
              Eli Collins
              Reporter:
              Eli Collins
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development