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. Text File
            mr-2377-20.txt
            1 kB
            Todd Lipcon
          2. Text File
            mr-2377-0.22.patch
            1 kB
            Konstantin Shvachko

            Issue Links

            is depended upon by

            Task - A task that needs to be done. HADOOP-8357 Restore security in Hadoop 0.22 branch

            • Major - Major loss of function.
            • Resolved
            is duplicated by

            Bug - A problem which impairs or prevents the functions of the product. HADOOP-7476 task-controller can drop last char from config file

            • Trivial - Cosmetic problem like misspelt words or misaligned text.
            • Resolved

              Activity

              Descending order - Click to sort in ascending order
              Hide
              Permalink
              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
              Permalink
              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
              Permalink
              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
              Permalink
              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
              Permalink
              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
              Permalink
              Benoy Antony added a comment -

              Patch for 0.22

              Show
              Benoy Antony added a comment - Patch for 0.22
              Hide
              Permalink
              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
              Permalink
              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
              Permalink
              Eli Collins added a comment -

              +1 lgtm

              Show
              Eli Collins added a comment - +1 lgtm
              Hide
              Permalink
              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
              Permalink
              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