kuenishi commented on PR #3225: URL: https://github.com/apache/ozone/pull/3225#issuecomment-1365604750
@sumitagrawl My solution for the recursive directory deletion would be to add volume id and bucket id to the protobuf message `DirectoryInfo`. Both versions may exist in the deleted directory table, but we can decode them correctly: If the key has `/volumeId/bucketId/...` format, then it's old version. If it doesn't, it must have non-zero volume id and bucket id. Directory deletion service will propagate the volume and bucket ids recursively, in case old directories with old formats. But in that way, old versions of Ozone does not work when it's downgraded and see unknown format of directory info. So my suggestion would be to create another table for recursively-deleted directories and put deleted directory infos of new format there. Then we'll need another directory deletion service, but we can work on it. I opened [a new issue for that](https://issues.apache.org/jira/browse/HDDS-7712), and will update my design there if it's ok for you. -- 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]
