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

Minor cleanup to UGI use in AdminService

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 2.8.0
    • Fix Version/s: 2.9.0, 3.0.0-alpha2
    • Component/s: resourcemanager
    • Labels:
      None

      Description

      Instead of calling UserGroupInformation.getCurrentUser() over and over, we should just use the stored daemonUser.

      1. YARN-4998.001.patch
        2 kB
        Daniel Templeton
      2. YARN-4998.002.patch
        2 kB
        Daniel Templeton

        Activity

        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10738 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10738/)
        YARN-4998. Minor cleanup to UGI use in AdminService. (Daniel Templeton (kasha: rev 733aa993134ba324c712590fa92b8ef230b0839a)

        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/AdminService.java
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10738 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10738/ ) YARN-4998 . Minor cleanup to UGI use in AdminService. (Daniel Templeton (kasha: rev 733aa993134ba324c712590fa92b8ef230b0839a) (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/AdminService.java
        Hide
        kasha Karthik Kambatla added a comment -

        +1, checking this in..

        Show
        kasha Karthik Kambatla added a comment - +1, checking this in..
        Hide
        templedf Daniel Templeton added a comment -

        Rebased

        Show
        templedf Daniel Templeton added a comment - Rebased
        Hide
        kasha Karthik Kambatla added a comment -

        Trivial change, +1. Unfortunately though, the patch does not apply any more.

        Daniel Templeton - mind revving it and submitting patch for Jenkins run?

        Show
        kasha Karthik Kambatla added a comment - Trivial change, +1. Unfortunately though, the patch does not apply any more. Daniel Templeton - mind revving it and submitting patch for Jenkins run?

          People

          • Assignee:
            templedf Daniel Templeton
            Reporter:
            templedf Daniel Templeton
          • Votes:
            1 Vote for this issue
            Watchers:
            7 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development