[HADOOP-14172] S3Guard: import does not import empty directory

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: HADOOP-13345
• Component/s: fs/s3
• Hadoop Flags: Reviewed
• Assignee: Sean Mackrory
• Reporter: Sean Mackrory
• Created: 10/Mar/17 23:58
• Updated: 06/Apr/17 21:13
• Resolved: 06/Apr/17 21:13

Description

It imports everything comes up from listFiles, which includes only files (and their parent directories as a side-effect). My first thought on doing this would be to override S3AFileSystem to add an optional parameter to use AcceptAllButSelfAndS3nDirs instead of AcceptFilesOnly. But we could also manually traverse the tree to get all FileStatus objects directory by directory like we do for diff. That's far slower but doesn't add surface area to S3AFileSystem. But there's also the impact to other S3 clients to worry about - I could go either way on that.

Issue Links

• is related to: HADOOP-14266 S3Guard: S3AFileSystem::listFiles() to employ MetadataStore