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

FairScheduler: PoolSchedulable#updateDemand() has potential redundant computation

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.0.3
    • Fix Version/s: 1.2.0
    • Component/s: contrib/fair-share
    • Labels:
      None
    • Hadoop Flags:
      Reviewed
    1. MR-4556.patch
      1 kB
      Karthik Kambatla
    2. MR-4556.patch
      1 kB
      Karthik Kambatla

      Issue Links

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        3h 26m 1 Karthik Kambatla (Inactive) 14/Aug/12 23:10
        In Progress In Progress Patch Available Patch Available
        53m 17s 1 Karthik Kambatla (Inactive) 15/Aug/12 00:04
        Patch Available Patch Available Resolved Resolved
        51d 52m 1 Alejandro Abdelnur 05/Oct/12 00:56
        Resolved Resolved Closed Closed
        222d 5h 19m 1 Matt Foley 15/May/13 06:16
        Gavin made changes -
        Reporter Karthik Kambatla [ kkambatl ] Karthik Kambatla [ kasha ]
        Gavin made changes -
        Assignee Karthik Kambatla [ kkambatl ] Karthik Kambatla [ kasha ]
        Matt Foley made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        Matt Foley added a comment -

        Closed upon release of Hadoop 1.2.0.

        Show
        Matt Foley added a comment - Closed upon release of Hadoop 1.2.0.
        Alejandro Abdelnur made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Hadoop Flags Reviewed [ 10343 ]
        Fix Version/s 1.2.0 [ 12321661 ]
        Resolution Fixed [ 1 ]
        Hide
        Alejandro Abdelnur added a comment -

        Thanks Karthik. Committed to branch-1.

        Show
        Alejandro Abdelnur added a comment - Thanks Karthik. Committed to branch-1.
        Hide
        Alejandro Abdelnur added a comment -

        +1 pending on YARN-23 to get in trunk.

        Show
        Alejandro Abdelnur added a comment - +1 pending on YARN-23 to get in trunk.
        Hide
        Hadoop QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12544157/MR-4556.patch
        against trunk revision .

        -1 patch. The patch command could not apply the patch.

        Console output: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/2827//console

        This message is automatically generated.

        Show
        Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12544157/MR-4556.patch against trunk revision . -1 patch. The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/2827//console This message is automatically generated.
        Karthik Kambatla (Inactive) made changes -
        Attachment MR-4556.patch [ 12544157 ]
        Hide
        Karthik Kambatla (Inactive) added a comment -

        Rebased on branch-1.

        Testing:
        TestFairScheduler#testPoolAssignment() and TestFairScheduler#testSetPoolTwice() verify PoolSchedulable#updateDemand() that the patch changes.

        However, a couple of other tests fail in TestFairScheduler. Will create a JIRA for those failures.

        Show
        Karthik Kambatla (Inactive) added a comment - Rebased on branch-1. Testing: TestFairScheduler#testPoolAssignment() and TestFairScheduler#testSetPoolTwice() verify PoolSchedulable#updateDemand() that the patch changes. However, a couple of other tests fail in TestFairScheduler . Will create a JIRA for those failures.
        Hide
        Hadoop QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12540965/MR-4556.patch
        against trunk revision .

        -1 patch. The patch command could not apply the patch.

        Console output: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/2727//console

        This message is automatically generated.

        Show
        Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12540965/MR-4556.patch against trunk revision . -1 patch. The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/2727//console This message is automatically generated.
        Karthik Kambatla (Inactive) made changes -
        Status In Progress [ 3 ] Patch Available [ 10002 ]
        Karthik Kambatla (Inactive) made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Karthik Kambatla (Inactive) made changes -
        Attachment MR-4556.patch [ 12540965 ]
        Hide
        Karthik Kambatla (Inactive) added a comment -

        MR1 side of YARN-23

        Show
        Karthik Kambatla (Inactive) added a comment - MR1 side of YARN-23
        Karthik Kambatla (Inactive) made changes -
        Field Original Value New Value
        Link This issue is a clone of YARN-23 [ YARN-23 ]
        Karthik Kambatla (Inactive) created issue -

          People

          • Assignee:
            Karthik Kambatla
            Reporter:
            Karthik Kambatla
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development