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

task-controller fails to parse configuration if it doesn't end in \n

    Details

    • Hadoop Flags:
      Reviewed
    • Target Version/s:

      Description

      If the task-controller.cfg file doesn't end in a newline, it fails to parse properly.

      1. mr-2377-20.txt
        1 kB
        Todd Lipcon
      2. mr-2377-0.22.patch
        1 kB
        Konstantin Shvachko

        Issue Links

          Activity

          Hide
          Matt Foley added a comment -

          Closed upon release of Hadoop-1.1.0.

          Show
          Matt Foley added a comment - Closed upon release of Hadoop-1.1.0.
          Hide
          Todd Lipcon added a comment -

          Verified this bug is not present in the MR2 container executor, so marked as resolved.

          Show
          Todd Lipcon added a comment - Verified this bug is not present in the MR2 container executor, so marked as resolved.
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-22-branch #104 (See https://builds.apache.org/job/Hadoop-Mapreduce-22-branch/104/)
          MAPREDUCE-2377. task-controller fails to parse configuration if it doesn't end in \n. Contributed by Todd Lipcon and Benoy Antony. (Revision 1346251)

          Result = SUCCESS
          shv : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1346251
          Files :

          • /hadoop/common/branches/branch-0.22/mapreduce/CHANGES.txt
          • /hadoop/common/branches/branch-0.22/mapreduce/src/c++/task-controller/impl/configuration.c
          • /hadoop/common/branches/branch-0.22/mapreduce/src/c++/task-controller/test/test-task-controller.c
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-22-branch #104 (See https://builds.apache.org/job/Hadoop-Mapreduce-22-branch/104/ ) MAPREDUCE-2377 . task-controller fails to parse configuration if it doesn't end in \n. Contributed by Todd Lipcon and Benoy Antony. (Revision 1346251) Result = SUCCESS shv : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1346251 Files : /hadoop/common/branches/branch-0.22/mapreduce/CHANGES.txt /hadoop/common/branches/branch-0.22/mapreduce/src/c++/task-controller/impl/configuration.c /hadoop/common/branches/branch-0.22/mapreduce/src/c++/task-controller/test/test-task-controller.c
          Hide
          Konstantin Shvachko added a comment -

          I just committed this to branch 0.22.1. Thank you Todd and Benoy.
          Will close it if not needed in other versions.

          Show
          Konstantin Shvachko added a comment - I just committed this to branch 0.22.1. Thank you Todd and Benoy. Will close it if not needed in other versions.
          Hide
          Konstantin Shvachko added a comment -

          Attachments to MAPREDUCE-2377 and MAPREDUCE-2376 were intermixed. Attaching the right one.

          Show
          Konstantin Shvachko added a comment - Attachments to MAPREDUCE-2377 and MAPREDUCE-2376 were intermixed. Attaching the right one.
          Hide
          Benoy Antony added a comment -

          Patch for 0.22

          Show
          Benoy Antony added a comment - Patch for 0.22
          Hide
          Eli Collins added a comment -

          Forgot to mention, aside from the test I also tested a build from a tarball running a couple MR jobs using the LTC.

          Show
          Eli Collins added a comment - Forgot to mention, aside from the test I also tested a build from a tarball running a couple MR jobs using the LTC.
          Hide
          Eli Collins added a comment -

          I've committed to branch-20-security. Leaving open for 22.

          Show
          Eli Collins added a comment - I've committed to branch-20-security. Leaving open for 22.
          Hide
          Eli Collins added a comment -

          +1 lgtm

          Show
          Eli Collins added a comment - +1 lgtm
          Hide
          Todd Lipcon added a comment -

          Patch against secure 0.20 (waiting on MR-2178 for trunk)

          Show
          Todd Lipcon added a comment - Patch against secure 0.20 (waiting on MR-2178 for trunk)
          Hide
          Todd Lipcon added a comment -

          This is for the task-controller after MAPREDUCE-2178

          Show
          Todd Lipcon added a comment - This is for the task-controller after MAPREDUCE-2178

            People

            • Assignee:
              Benoy Antony
              Reporter:
              Todd Lipcon
            • Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development