swagle commented on issue #178: HDDS-2498. Fix sonar issues found in StorageContainerManager. URL: https://github.com/apache/hadoop-ozone/pull/178#issuecomment-555161021 I've added a TODO because: 1. The synchronization will not protect double decrement, seems wrongly intended locking behavior. 2. IMO removal listener will also be triggered for cache invalidation since we are changing metric values need to gather more context to figure out what was the cause and effect here.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
