Details
-
Improvement
-
Status: Closed
-
Major
-
Resolution: Fixed
-
None
-
None
-
On contrib/raid, the RaidNode currently runs the deletion check for parity files on directories too. It runs okay because the directory is not empty and trying to delete it non-recursively fails, but such failure messages only polute the log file.
My proposal is the following:
If recursePurge is checking a directory, it should call itself recursively.
If it's checking a file, it should do the deletion check.On contrib/raid, the RaidNode currently runs the deletion check for parity files on directories too. It runs okay because the directory is not empty and trying to delete it non-recursively fails, but such failure messages only polute the log file. My proposal is the following: If recursePurge is checking a directory, it should call itself recursively. If it's checking a file, it should do the deletion check.
-
Reviewed
Attachments
Attachments
Issue Links
- depends upon
-
-
- Closed
-