fapifta commented on PR #6990: URL: https://github.com/apache/ozone/pull/6990#issuecomment-2253561739
@kerneltime yes, I absolutely agree, here is what is in my head currently: This change is not really about getting rid of all non-optimal edge cases, the aim is to get rid of things that are easy to get rid of. My secondary goal is to gain experience in how our client is implemented, as I think we should not have any checks done on the client side that we can do only by doing an RPC. At the end of the day I want to play with the idea of removing most of the checks from the client side and making it really thin, because, to be honest, we have to do these checks anyway on the server side also. All in all, once the easily removable excess RPCs are done, I am planning to come up with some larger re-design in this direction. I am not sure if you had the time to check into the new changes, I would be glad to be sure that you still approve the current version. -- 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]
