XMLWordPrintableJSON

    Details

    • Type: Sub-task
    • Status: Patch Available
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 2.8.1
    • Fix Version/s: None
    • Component/s: fs/s3
    • Labels:
      None

      Description

      The fix to avoid calls to getFileStatus() for each path component in deleteUnnecessaryFakeDirectories() (HADOOP-13164) results in accumulation of delete markers in versioned S3 buckets. The above patch replaced getFileStatus() checks with a single batch delete request formed by generating all ancestor keys formed from a given path. Since the delete request is not checking for existence of fake directories, it will create a delete marker for every path component that did not exist (or was previously deleted). Note that issuing a DELETE request without specifying a version ID will always create a new delete marker, even if one already exists (AWS S3 Developer Guide)

      Since deleteUnnecessaryFakeDirectories() is called as a callback on successful writes and on renames, delete markers accumulate rather quickly and their rate of accumulation is inversely proportional to the depth of the path. In other words, directories closer to the root will have more delete markers than the leaves.

      This behavior negatively impacts performance of getFileStatus() operation when it has to issue listObjects() request (especially v1) as the delete markers have to be examined when the request searches for first current non-deleted version of an object following a given prefix.

      I did a quick comparison against 3.x and the issue is still present: https://github.com/apache/hadoop/blob/trunk/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/S3AFileSystem.java#L2947

       

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                stevel@apache.org Steve Loughran
                Reporter:
                dchmelev Dmitri Chmelev
              • Votes:
                1 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated: