Hadoop HDFS
  1. Hadoop HDFS
  2. HDFS-1096

allow dfsadmin/mradmin refresh of superuser proxy group mappings

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.22.0
    • Fix Version/s: 0.22.0
    • Component/s: security
    • Labels:
      None
    • Hadoop Flags:
      Reviewed
    • Release Note:
      Hide
      changed protocol name (may be used in hadoop-policy.xml)
      from security.refresh.usertogroups.mappings.protocol.acl to security.refresh.user.mappings.protocol.acl
      Show
      changed protocol name (may be used in hadoop-policy.xml) from security.refresh.usertogroups.mappings.protocol.acl to security.refresh.user.mappings.protocol.acl
    • Tags:
      security
    1. ASF.LICENSE.NOT.GRANTED--HDFS-1096-BP20-4.patch
      50 kB
      Boris Shkolnik
    2. ASF.LICENSE.NOT.GRANTED--HDFS-1096-BP20-6.patch
      49 kB
      Boris Shkolnik
    3. ASF.LICENSE.NOT.GRANTED--HDFS-1096-BP20-6-fix.patch
      0.7 kB
      Boris Shkolnik
    4. HDFS-1096-BP20-7.patch
      49 kB
      Boris Shkolnik
    5. HDFS-1096-3.patch
      26 kB
      Boris Shkolnik
    6. HDFS-1096-fix.patch
      1 kB
      Boris Shkolnik

      Issue Links

        Activity

        Hide
        Boris Shkolnik added a comment -

        for previous version, not for commit.

        Show
        Boris Shkolnik added a comment - for previous version, not for commit.
        Hide
        Jitendra Nath Pandey added a comment -

        I think the static hash-maps proxyHosts and proxyGroups should be protected for multi-thread safety.

        Apart from that looks good.

        Show
        Jitendra Nath Pandey added a comment - I think the static hash-maps proxyHosts and proxyGroups should be protected for multi-thread safety. Apart from that looks good.
        Hide
        Boris Shkolnik added a comment -

        addressing comments

        Show
        Boris Shkolnik added a comment - addressing comments
        Hide
        Boris Shkolnik added a comment -

        fix for the patch

        Show
        Boris Shkolnik added a comment - fix for the patch
        Hide
        Boris Shkolnik added a comment -

        combined the fix into one patch for prev version.

        Show
        Boris Shkolnik added a comment - combined the fix into one patch for prev version.
        Hide
        Boris Shkolnik added a comment -

        ran test-patch and test manually.

        Show
        Boris Shkolnik added a comment - ran test-patch and test manually.
        Hide
        Jitendra Nath Pandey added a comment -

        +1 for the patch.

        Show
        Jitendra Nath Pandey added a comment - +1 for the patch.
        Hide
        Boris Shkolnik added a comment -

        committed to trunk

        Show
        Boris Shkolnik added a comment - committed to trunk
        Hide
        Boris Shkolnik added a comment -

        One of the files still had RefreshUserGroupsMappingsProtocol not replaced with RefreshUserMappingsProtocol.

        Show
        Boris Shkolnik added a comment - One of the files still had RefreshUserGroupsMappingsProtocol not replaced with RefreshUserMappingsProtocol.
        Hide
        Jakob Homan added a comment -

        +1 on hdfs-109-fix.patch. Fixes the build for me.

        Show
        Jakob Homan added a comment - +1 on hdfs-109-fix.patch. Fixes the build for me.
        Hide
        Hudson added a comment -

        Integrated in Hadoop-Hdfs-trunk-Commit #298 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Hdfs-trunk-Commit/298/)

        Show
        Hudson added a comment - Integrated in Hadoop-Hdfs-trunk-Commit #298 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Hdfs-trunk-Commit/298/ )
        Hide
        Tsz Wo Nicholas Sze added a comment -

        Hi Boris,

        • new commit should be appended at end of the list in CHANGE.txt but not inserted in the beginning.
        • you don't now have to add new item in CHANGE.txt for fixing previous commit.
              HDFS-1096. fix for prev. commit. (boryas)
          
              HDFS-1096. allow dfsadmin/mradmin refresh of superuser proxy group
               mappings (boryas)
          
        Show
        Tsz Wo Nicholas Sze added a comment - Hi Boris, new commit should be appended at end of the list in CHANGE.txt but not inserted in the beginning. you don't now have to add new item in CHANGE.txt for fixing previous commit. HDFS-1096. fix for prev. commit. (boryas) HDFS-1096. allow dfsadmin/mradmin refresh of superuser proxy group mappings (boryas)
        Hide
        Boris Shkolnik added a comment -

        ok.

        Show
        Boris Shkolnik added a comment - ok.

          People

          • Assignee:
            Boris Shkolnik
            Reporter:
            Boris Shkolnik
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development