Uploaded image for project: 'Hadoop YARN'
  1. Hadoop YARN
  2. YARN-7069 [Umbrella] YARN Resource Profiles phase#2 enhancements
  3. YARN-6953

Clean up ResourceUtils.setMinimumAllocationForMandatoryResources() and setMaximumAllocationForMandatoryResources()

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • YARN-3926
    • 3.1.0, 3.0.1
    • resourcemanager
    • Reviewed

    Description

      The setMinimumAllocationForMandatoryResources() and setMaximumAllocationForMandatoryResources() methods are quite convoluted. They'd be much simpler if they just handled CPU and memory manually instead of trying to be clever about doing it in a loop. There are also issues, such as the log warning always talking about memory or the last element of the inner array being a copy of the first element.

      Attachments

        1. YARN-6953.009.patch
          7 kB
          Manikandan R
        2. YARN-6953.008.patch
          7 kB
          Manikandan R
        3. YARN-6953.007.patch
          7 kB
          Manikandan R
        4. YARN-6953-YARN-3926.006.patch
          7 kB
          Manikandan R
        5. YARN-6953-YARN-3926.005.patch
          7 kB
          Manikandan R
        6. YARN-6953-YARN-3926-WIP.patch
          8 kB
          Manikandan R
        7. YARN-6953-YARN-3926.004.patch
          6 kB
          Manikandan R
        8. YARN-6953-YARN-3926.003.patch
          6 kB
          Manikandan R
        9. YARN-6953-YARN-3926.002.patch
          6 kB
          Manikandan R
        10. YARN-6953-YARN-3926.001.patch
          7 kB
          Manikandan R

        Activity

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

          People

            manirajv06@gmail.com Manikandan R
            templedf Daniel Templeton
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Issue deployment