Details

      Description

      The current serialization of the ACLs is suboptimal. ACL entries should be serialized using the same scheme that the PB-based FSImage serializes permissions.

      An ACL entry is represented by a 32-bit integer in Big Endian format. The bits can be divided in four segments:
      [0:2) || [2:26) || [26:27) || [27:29) || [29:32)

      [0:2) – reserved for futute uses.
      [2:26) – the name of the entry, which is an ID that points to a string in the StringTableSection.
      [26:27) – the scope of the entry (AclEntryScopeProto)
      [27:29) – the type of the entry (AclEntryTypeProto)
      [29:32) – the permission of the entry (FsActionProto)

      1. HDFS-5933.002.patch
        19 kB
        Haohui Mai
      2. HDFS-5933.001.patch
        22 kB
        Haohui Mai
      3. HDFS-5933.000.patch
        22 kB
        Haohui Mai

        Issue Links

          Activity

          Hide
          Haohui Mai added a comment -

          The v1 patch fixes the trailing whitespace.

          Show
          Haohui Mai added a comment - The v1 patch fixes the trailing whitespace.
          Hide
          Chris Nauroth added a comment -

          +1 for this patch, pending commit of HDFS-5923 first.

          Show
          Chris Nauroth added a comment - +1 for this patch, pending commit of HDFS-5923 first.
          Hide
          Haohui Mai added a comment -

          Rebase on the HDFS-5922 v3 patch

          Show
          Haohui Mai added a comment - Rebase on the HDFS-5922 v3 patch
          Hide
          Chris Nauroth added a comment -

          +1 for the v2 rebase patch. I committed it to the HDFS-4685 branch. Thanks again, Haohui.

          Show
          Chris Nauroth added a comment - +1 for the v2 rebase patch. I committed it to the HDFS-4685 branch. Thanks again, Haohui.

            People

            • Assignee:
              Haohui Mai
              Reporter:
              Haohui Mai
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development