Hadoop HDFS
  1. Hadoop HDFS
  2. HDFS-4919

Improve documentation of dfs.permissions.enabled flag.

    Details

    • Type: Improvement Improvement
    • Status: Patch Available
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 3.0.0, 2.1.0-beta
    • Fix Version/s: None
    • Component/s: documentation
    • Labels:
      None

      Description

      The description of dfs.permissions.enabled in hdfs-default.xml does not state that permissions are always checked on certain calls regardless of this configuration. The HDFS permissions guide still mentions the deprecated dfs.permissions property instead of the currently supported dfs.permissions.enabled.

      http://hadoop.apache.org/docs/current/hadoop-project-dist/hadoop-hdfs/HdfsPermissionsGuide.html#Configuration_Parameters

      1. HDFS-4919.patch
        1 kB
        Shinichi Yamashita

        Issue Links

          Activity

          Chris Nauroth created issue -
          Chris Nauroth made changes -
          Field Original Value New Value
          Link This issue relates to HDFS-4918 [ HDFS-4918 ]
          Hide
          Colin Patrick McCabe added a comment -

          Do we still need this flag in hadoop-2 and beyond? The documentation implies that part of having it was to "allow administrators to reliably set owners and permissions in advance of turning on regular permissions checking." Surely everyone who needed to do this conversion process has already done so since hadoop 0.22. I don't know of anyone who would use this flag nowadays, even developers.

          Show
          Colin Patrick McCabe added a comment - Do we still need this flag in hadoop-2 and beyond? The documentation implies that part of having it was to "allow administrators to reliably set owners and permissions in advance of turning on regular permissions checking." Surely everyone who needed to do this conversion process has already done so since hadoop 0.22. I don't know of anyone who would use this flag nowadays, even developers.
          Hide
          Suresh Srinivas added a comment -

          Surely everyone who needed to do this conversion process has already done so since hadoop 0.22.

          Clearly some folks reporting HDFS-4918/commenting on it are using this. In the interest of incompatible config changes, lets leave it alone. But documenting it would be good.

          Show
          Suresh Srinivas added a comment - Surely everyone who needed to do this conversion process has already done so since hadoop 0.22. Clearly some folks reporting HDFS-4918 /commenting on it are using this. In the interest of incompatible config changes, lets leave it alone. But documenting it would be good.
          Hide
          Shinichi Yamashita added a comment -

          Hi,
          The explanation of "dfs.web.ugi" seems to be old as well as "dfs.permissions". It should revise both.

          Show
          Shinichi Yamashita added a comment - Hi, The explanation of "dfs.web.ugi" seems to be old as well as "dfs.permissions". It should revise both.
          Hide
          Shinichi Yamashita added a comment -

          There are some cases using these properties. And the property name in the document should be accurate.
          I attach the patch which I changed to a current property name.

          Show
          Shinichi Yamashita added a comment - There are some cases using these properties. And the property name in the document should be accurate. I attach the patch which I changed to a current property name.
          Shinichi Yamashita made changes -
          Attachment HDFS-4919.patch [ 12600936 ]
          Shinichi Yamashita made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Target Version/s 3.0.0, 2.1.0-beta [ 12320356, 12324031 ] 2.1.0-beta, 3.0.0 [ 12324031, 12320356 ]

            People

            • Assignee:
              Unassigned
              Reporter:
              Chris Nauroth
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:

                Development