Hadoop Map/Reduce
  1. Hadoop Map/Reduce
  2. MAPREDUCE-2304

TestMRCLI fails when hostname has a hyphen (-)

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.22.0
    • Fix Version/s: 0.22.0
    • Component/s: test
    • Labels:
      None
    • Environment:

      Linux

    • Hadoop Flags:
      Reviewed

      Description

      TestMRCLI fails with below

      Comparator: [RegexpComparator]
      Comparision result: [fail]
      Expected output: [mv: Wrong FS: har:/dest/dir0.har/dir0/file0, expected: hdfs://\w+[.a-z]*:[0-9]+]
      Actual output: [mv: Wrong FS: har:/dest/dir0.har/dir0/file0, expected: hdfs://lab-something.host.com:34039

      1. MAPREDUCE-2304.patch
        0.7 kB
        Priyo Mustafi

        Issue Links

          Activity

          Priyo Mustafi created issue -
          Hide
          Priyo Mustafi added a comment -

          The host can and have numeric. Interestingly the test works on windows without the patch

          Show
          Priyo Mustafi added a comment - The host can and have numeric. Interestingly the test works on windows without the patch
          Priyo Mustafi made changes -
          Field Original Value New Value
          Attachment MAPREDUCE-2304.patch [ 12470275 ]
          Jakob Homan made changes -
          Summary TestMRCLI fails when hostname has a hiphen (-) TestMRCLI fails when hostname has a hyphen (-)
          Hide
          Konstantin Shvachko added a comment -

          It probably has to do something with configuration files on this machine you ran into the issue. Because if you run tests with the default configuration only, then you should just have localhost or 0.0.0.0 as a hostname. But it's a good catch, and extends the applicability of the test.
          +1

          Show
          Konstantin Shvachko added a comment - It probably has to do something with configuration files on this machine you ran into the issue. Because if you run tests with the default configuration only, then you should just have localhost or 0.0.0.0 as a hostname. But it's a good catch, and extends the applicability of the test. +1
          Hide
          Konstantin Shvachko added a comment -

          I just committed this. Thank you Priyo.

          Show
          Konstantin Shvachko added a comment - I just committed this. Thank you Priyo.
          Konstantin Shvachko made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Hadoop Flags [Reviewed]
          Fix Version/s 0.22.0 [ 12314184 ]
          Resolution Fixed [ 1 ]
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk-Commit #604 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-trunk-Commit/604/)
          MAPREDUCE-2304. Fix TestMRCLI to allow hostname with a hyphen . Contributed by Priyo Mustafi.

          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk-Commit #604 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-trunk-Commit/604/ ) MAPREDUCE-2304 . Fix TestMRCLI to allow hostname with a hyphen . Contributed by Priyo Mustafi.
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-22-branch #33 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-22-branch/33/)

          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-22-branch #33 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-22-branch/33/ )
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk #643 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-trunk/643/)

          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk #643 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-trunk/643/ )
          Konstantin Shvachko made changes -
          Link This issue relates to HDFS-2232 [ HDFS-2232 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          4h 31m 1 Konstantin Shvachko 04/Feb/11 23:36

            People

            • Assignee:
              Priyo Mustafi
              Reporter:
              Priyo Mustafi
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development