bosschaert commented on pull request #13: URL: https://github.com/apache/sling-org-apache-sling-feature-apiregions/pull/13#issuecomment-656736499
The Sonar bug reported says on the field featureRegionMap: "Use a thread-safe type; adding "volatile" is not enough to make this field thread-safe." However the value of this field is an immutable map with immutable content which is entirely replaced each time it's modified, so I think Sonar's assessment is wrong. Also its strange that it only picks up on this field, as bsnVerMap, bundleFeatureMap and regionPackageMap follow the same pattern. ---------------------------------------------------------------- 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]
