Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-9026

Hadoop.cmd fails to initialize if user's %path% variable has parenthesis

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 1-win
    • 1-win
    • None
    • None
    • Reviewed

    Description

      Hadoop.cmd fails to initialize if user's %path% variable has parenthesis.
      This happens in "updatepath" module while tring the add Hadoop_bin_path to %path%
      Repro:
      1. Create a folder C:\random()
      2. Add this path to %path%
      3. Start Hadoop command line

      Error:
      "<path_after_c:\rand()>"; unexpected at this time

      Reported by Ramya Nimmagadda.

      Attachments

        Issue Links

          Activity

            ivanmi Ivan Mitic added a comment -

            Attaching a proposal for a fix.

            ivanmi Ivan Mitic added a comment - Attaching a proposal for a fix.
            hadoopqa Hadoop QA added a comment -

            -1 overall. Here are the results of testing the latest attachment
            http://issues.apache.org/jira/secure/attachment/12553227/HADOOP-9026.branch-1-win.pathfix.patch
            against trunk revision .

            -1 patch. The patch command could not apply the patch.

            Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/1733//console

            This message is automatically generated.

            hadoopqa Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12553227/HADOOP-9026.branch-1-win.pathfix.patch against trunk revision . -1 patch . The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/1733//console This message is automatically generated.
            chuanliu Chuan Liu added a comment -

            Do we still need to remove '(x86)' in path after this change?

            chuanliu Chuan Liu added a comment - Do we still need to remove '(x86)' in path after this change?
            ivanmi Ivan Mitic added a comment -

            Do we still need to remove '(x86)' in path after this change?

            Good question Chuan. I actually think that we do not. I'll update the patch.

            ivanmi Ivan Mitic added a comment - Do we still need to remove '(x86)' in path after this change? Good question Chuan. I actually think that we do not. I'll update the patch.
            ivanmi Ivan Mitic added a comment -

            Attaching updated patch.

            ivanmi Ivan Mitic added a comment - Attaching updated patch.
            chuanliu Chuan Liu added a comment -

            +1 Thanks for the fix!

            chuanliu Chuan Liu added a comment - +1 Thanks for the fix!
            hadoopqa Hadoop QA added a comment -

            -1 overall. Here are the results of testing the latest attachment
            http://issues.apache.org/jira/secure/attachment/12553435/HADOOP-9026.branch-1-win.pathfix.2.patch
            against trunk revision .

            -1 patch. The patch command could not apply the patch.

            Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/1739//console

            This message is automatically generated.

            hadoopqa Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12553435/HADOOP-9026.branch-1-win.pathfix.2.patch against trunk revision . -1 patch . The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/1739//console This message is automatically generated.
            chuanliu Chuan Liu added a comment -

            Hello, I am traveling and can only check my emails sporadically. I will respond to your message on or after Nov 23, 2012 (if not sooner). Thanks.

            chuanliu Chuan Liu added a comment - Hello, I am traveling and can only check my emails sporadically. I will respond to your message on or after Nov 23, 2012 (if not sooner). Thanks.

            +1. Committed the patch to branch-1-win.

            Thank you Ivan.

            sureshms Suresh Srinivas added a comment - +1. Committed the patch to branch-1-win. Thank you Ivan.

            People

              ivanmi Ivan Mitic
              ivanmi Ivan Mitic
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: