cchung100m commented on PR #8990: URL: https://github.com/apache/ozone/pull/8990#issuecomment-3249561506
Hi @sodonnel Thanks for the prompt reply and suggestions. I can update the part you mentioned. Hi @peterxcli Any thoughts would be appreciated if you are available. 😄 >However, what I am not sure about, is whether any of the caller of getReplicas() would assume they have their own copy they can modify, and therefore that code would fail. So while this would be efficient, it really constrains how the return value can be used, so it may be too risky to change. -- 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]
