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

MR-279: Metrics for reserved resource in ResourceManager

    Details

    • Type: New Feature
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.23.0
    • Fix Version/s: 0.23.0
    • Component/s: mrv2
    • Labels:
      None
    • Tags:
      metrics,mrv2,rm

      Description

      Add metrics for reserved resources.

        Activity

        Hide
        vicaya Luke Lu added a comment -

        Arun: review the reserved resource logic?

        Show
        vicaya Luke Lu added a comment - Arun: review the reserved resource logic?
        Hide
        acmurthy Arun C Murthy added a comment -

        Seems ok - just to clarify: You've added Node.getReservedResource just to get the metric back during Application.unreserve, correct?

        Show
        acmurthy Arun C Murthy added a comment - Seems ok - just to clarify: You've added Node.getReservedResource just to get the metric back during Application.unreserve, correct?
        Hide
        vicaya Luke Lu added a comment -

        You've added Node.getReservedResource just to get the metric back during Application.unreserve, correct?

        Yes. The NodeManagerImpl seems to be the place to save it as it also talks about update the reserved resource etc. but not really save it

        Show
        vicaya Luke Lu added a comment - You've added Node.getReservedResource just to get the metric back during Application.unreserve, correct? Yes. The NodeManagerImpl seems to be the place to save it as it also talks about update the reserved resource etc. but not really save it
        Hide
        acmurthy Arun C Murthy added a comment -

        +1, lgtm

        Show
        acmurthy Arun C Murthy added a comment - +1, lgtm
        Hide
        acmurthy Arun C Murthy added a comment -

        I just committed this. Thanks Luke!

        Show
        acmurthy Arun C Murthy added a comment - I just committed this. Thanks Luke!

          People

          • Assignee:
            vicaya Luke Lu
            Reporter:
            vicaya Luke Lu
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development