Details
-
Sub-task
-
Status: Resolved
-
Major
-
Resolution: Fixed
-
None
-
None
-
Reviewed
Description
This jira aims to fix two bugs in deleting files with snapshot:
1. We should update the quota usage when the file/dir to be deleted is not included in the latest snapshot (i.e., the to-be-deleted file/dir is recorded in the created list of the latest snapshot). In particular, in FSDirectory#removeLastINode, instead of checking if the latest snapshot is null to decide whether we should update the quota usage, we should use INode#isInLatestSnapshot(Snapshot) as the check condition.
2. Similarly, in FSDirectory#unprotectedDelete(...), we should update quota usage after calling INode#cleanSubtree, considering during cleanSubtree() we may delete files that were created after taking the latest snapshot.