whbing commented on PR #6127: URL: https://github.com/apache/ozone/pull/6127#issuecomment-1918912013
> as this `getState ` is fully synchronized method. We need to see if this method is really needed to be `synchronized`. I feel it is not need to be `synchronized`. Not sure to what extent it affects performance. At least in practice, we haven't encountered any bottlenecks caused by `synchronized getState` yet. Maybe we can revisit it in the future if the problem occurs ? -- 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]
