ChenSammi commented on PR #8909:
URL: https://github.com/apache/ozone/pull/8909#issuecomment-3204826755

   > @ChenSammi Do we currently have a logic to check whether a directory is 
empty before it is eligible to be deleted by the `KeyLifecycleService`? There 
might be cases where the parent directory modificationTime is expired but the 
children have not expired, but since the parent directory is expired and 
deleted, the children keys will also be deleted.
   
   Yeah, found the same problem. I plan to have a next patch to check and 
delete only keys, ignore the directories, so we will not incorrectly delete any 
files under a nested directory, where directory is expired according to it's 
modification time, but files are not.  The drawback is there will be a few 
empty directories left there.  Thoughts? 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to