duongnguyen0 commented on PR #3594: URL: https://github.com/apache/ozone/pull/3594#issuecomment-1188457979
> Thanks @duongnguyen0 for refining the patch. Can `ObjectEndpoint` use `ClientProtocol`/`RpcClient` directly, instead of having to pollute `OzoneVolume` with key-related operations? Updated to use ClientProtocol directly instead of putting delegation in `OzoneVolume`. I didn't clean up all delegations in `OzoneVolume` and `OzoneBucket`, thought, as they're used by OFS as well. And anyway, same optimization to avoid calling `getBucket` (or maybe even `getVolume` because volume is already in OFS path context) can be done in many OFS operations I believe. -- 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]
