Uploaded image for project: 'Hadoop YARN'
  1. Hadoop YARN
  2. YARN-9092

Create an object for cgroups mount enable and cgroups mount path as they belong together

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

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • None
    • 3.3.0, 3.2.1, 3.1.3
    • None
    • None
    • Reviewed

    Description

      YarnConfiguration.NM_LINUX_CONTAINER_CGROUPS_MOUNT and 

      YarnConfiguration.NM_LINUX_CONTAINER_CGROUPS_MOUNT_PATH are used in conjunction many places in the code, so for the sake of readabilty and simplicity, it is better to wrap the values of these configs to an object and use it instead of having 2 fields in 

      CGroupsHandlerImpl and in CgroupsLCEResourcesHandler as well.

      Attachments

        1. YARN-9092.003.patch
          10 kB
          Szilard Nemeth
        2. YARN-9092.002.patch
          10 kB
          Szilard Nemeth
        3. YARN-9092.001.patch
          10 kB
          Szilard Nemeth

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

          People

            shuzirra Gergely Pollák
            snemeth Szilard Nemeth
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment