Hadoop HDFS
  1. Hadoop HDFS
  2. HDFS-6134 Transparent data at rest encryption
  3. HDFS-6738

Remove unnecessary getEncryptionZoneForPath call in EZManager#createEncryptionZone

    Details

      Description

      We can remove the following getEncryptionZoneForPath call below.

      // done this way to handle edit log loading
          dir.unprotectedSetXAttrs(src, xattrs, EnumSet.of(XAttrSetFlag.CREATE));
          ezi = getEncryptionZoneForPath(srcIIP);
          return ezXAttr;
      

        Activity

        Charles Lamb made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Fix Version/s fs-encryption (HADOOP-10150 and HDFS-6134) [ 12326852 ]
        Resolution Fixed [ 1 ]
        Hide
        Charles Lamb added a comment -

        Thanks for the review Uma Maheswara Rao G. I've committed this to fs-encryption.

        Show
        Charles Lamb added a comment - Thanks for the review Uma Maheswara Rao G . I've committed this to fs-encryption.
        Hide
        Uma Maheswara Rao G added a comment -

        +1

        Show
        Uma Maheswara Rao G added a comment - +1
        Hide
        Hadoop QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12657363/HDFS-6738.001.patch
        against trunk revision .

        -1 patch. The patch command could not apply the patch.

        Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/7442//console

        This message is automatically generated.

        Show
        Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12657363/HDFS-6738.001.patch against trunk revision . -1 patch . The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/7442//console This message is automatically generated.
        Charles Lamb made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Target Version/s fs-encryption (HADOOP-10150 and HDFS-6134) [ 12326852 ]
        Hide
        Charles Lamb added a comment -

        Compilation issue only so no tests included in this patch.

        Show
        Charles Lamb added a comment - Compilation issue only so no tests included in this patch.
        Charles Lamb made changes -
        Attachment HDFS-6738.001.patch [ 12657363 ]
        Hide
        Charles Lamb added a comment -

        Thanks Uma Maheswara Rao G.

        The attached trivial patch addresses the issue.

        Show
        Charles Lamb added a comment - Thanks Uma Maheswara Rao G . The attached trivial patch addresses the issue.
        Uma Maheswara Rao G made changes -
        Field Original Value New Value
        Priority Major [ 3 ] Minor [ 4 ]
        Uma Maheswara Rao G created issue -

          People

          • Assignee:
            Uma Maheswara Rao G
            Reporter:
            Uma Maheswara Rao G
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development