slfan1989 commented on PR #7266: URL: https://github.com/apache/ozone/pull/7266#issuecomment-3173934334
> > Sorry for the delay. Can we have this PR be just the SCM <-> client communication for querying volume info? Right now it also contains information for SCM <-> DN communication about the failure time of the volume, which is not directly related and can be added in a follow-up change. The new RPC will also need tests added. > > @errose28 Thank you for your feedback. I will improve this PR based on your suggestions. @errose28 @adoroszlai Could you please take another look at this PR? Thank you very much! I’ve simplified it to only include the interactions between the Client and SCM. If you think the implementation meets expectations, I’ll add some unit tests. -- 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]
