Uploaded image for project: 'Hadoop Map/Reduce'
  1. Hadoop Map/Reduce
  2. MAPREDUCE-3419

Don't mark exited TT threads as dead in MiniMRCluster

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 1.1.0
    • 1.1.0
    • tasktracker, test
    • None
    • Reviewed

    Description

      MAPREDUCE-2850 flagged all TT threads that exited in the MiniMRCluster as dead, this breaks a number of the other tests that use MiniMRCluster across restart.

      Attachments

        1. mapreduce-2850-1.patch
          1 kB
          Eli Collins

        Issue Links

          Activity

            eli Eli Collins added a comment -

            The change to MiniMRCluster to clean up exited TTs threads is actually somewhat invasive so let's just revert the setting of isDead from MR-2850 and add a field that flags what TestDiskFailures would like to check, which is that the TT exited cleanly.

            eli Eli Collins added a comment - The change to MiniMRCluster to clean up exited TTs threads is actually somewhat invasive so let's just revert the setting of isDead from MR-2850 and add a field that flags what TestDiskFailures would like to check, which is that the TT exited cleanly.
            ravidotg Ravi Gummadi added a comment -

            Patch looks good to me.
            +1

            Hope you ran all tests that were failing earlier.

            ravidotg Ravi Gummadi added a comment - Patch looks good to me. +1 Hope you ran all tests that were failing earlier.
            eli Eli Collins added a comment -

            Thanks Ravi. Yea, this fixes the other tets. I'll re-run the whole suite on branch-20-security for sanity before committing.

            eli Eli Collins added a comment - Thanks Ravi. Yea, this fixes the other tets. I'll re-run the whole suite on branch-20-security for sanity before committing.
            eli Eli Collins added a comment -

            All tests ran cleanly except for one, which is unrelated, filed HADOOP-7836 for it. I've committed this.

            eli Eli Collins added a comment - All tests ran cleanly except for one, which is unrelated, filed HADOOP-7836 for it. I've committed this.
            mattf Matthew Foley added a comment -

            Closed upon release of Hadoop-1.1.0.

            mattf Matthew Foley added a comment - Closed upon release of Hadoop-1.1.0.

            People

              eli Eli Collins
              eli Eli Collins
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: