-
Notifications
You must be signed in to change notification settings - Fork 478
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HDDS-4495. [FSO]Delete : Implement async cleanup of garbage and orphan sub-dirs/files #2093
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Haven't taken a deep review for this, but some minor comments below.
hadoop-ozone/common/src/main/java/org/apache/hadoop/ozone/om/OMConfigKeys.java
Outdated
Show resolved
Hide resolved
...p-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/DirectoryDeletingService.java
Show resolved
Hide resolved
...ntegration-test/src/test/java/org/apache/hadoop/fs/ozone/TestDirectoryDeletingServiceV1.java
Outdated
Show resolved
Hide resolved
...ntegration-test/src/test/java/org/apache/hadoop/fs/ozone/TestDirectoryDeletingServiceV1.java
Outdated
Show resolved
Hide resolved
...ntegration-test/src/test/java/org/apache/hadoop/fs/ozone/TestDirectoryDeletingServiceV1.java
Outdated
Show resolved
Hide resolved
Thanks again @linyiqun for the review comments. I've updated PR by addressing the comments. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, +1.
Thanks for addressing the review comments!
Thanks a lot @linyiqun, I will merge it. |
What changes were proposed in this pull request?
Implements a service to get the pending orphan directories and clean up its sub-paths(subdirs and subfiles). Sub-files will be added to existing KeyDeletinService where it will ask scm to clean its blocks, is any.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-4495
How was this patch tested?
Added a few cases, will add more cases to it.
Raised followup jira to verify KDS HDDS-5042