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

JobImpl uber checks for cpu are wrong

VotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Critical
    • Resolution: Fixed
    • 2.0.3-alpha
    • 2.0.3-alpha
    • mrv2
    • None

    Description

      The cpu checks for uberizing have two issues:

      1. the defaults are hardcoded instead of using the conf defaults
      2. the comparison against the sys cpu size is using < instead of <=
          boolean smallCpu =
              (
                  Math.max(
                      conf.getInt(MRJobConfig.MAP_CPU_VCORES, 1), 
                      conf.getInt(MRJobConfig.REDUCE_CPU_VCORES, 1)) < 
                   sysCPUSizeForUberSlot
              );
      

      Everything is defaulting to 1, so uber cpu checks are now disabled causing TestUberAM to fail.

      Attachments

        1. MAPREDUCE-4936.patch
          1 kB
          Arun Murthy

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            acmurthy Arun Murthy
            daryn Daryn Sharp
            Votes:
            0 Vote for this issue
            Watchers:
            8 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment