bdelacretaz commented on pull request #25: URL: https://github.com/apache/sling-org-apache-sling-graphql-core/pull/25#issuecomment-866680040
Thank you for the detailed review, all good suggestions! Of course, HashMap not being synchronized it doesn't make sense to make the `LazyLoadingMap` thread safe, I have removed all synchronization in that class. I think I have implemented all other suggestions, please cross-check. -- 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]
