aswinshakil commented on PR #5337: URL: https://github.com/apache/ozone/pull/5337#issuecomment-1730354534
Looking at the code, I think it should be fine to use Active DB's `Key Manager`. Because the directory is already deleted, iterating over Active DB or Snapshot DB would yield the same result as we can't add or delete anything inside the directory. I'm not sure if there are any other side effects to using Active DB instead of Snapshot DB. But it is better to use Snapshot DB for this purpose so that we don't mess up anything on Active DB. -- 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]
