ivandika3 commented on PR #9157: URL: https://github.com/apache/ozone/pull/9157#issuecomment-3467773737
> That being said, a number of request types are also issued via OM Ratis directly (e.g. PurgeKeys, PurgeDirectories) but their handlers do record detailed OM lock information. @jojochuang Yes, I think these requests follow the current convention. IMO I'm OK if we go ahead with this change as a way to standardize the usage of locks (the overhead should not be that large). We can extract these locking methods instead. In the future, we might want to defer the background service in another process and the OM lock details. -- 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]
