justinedelson commented on pull request #20: URL: https://github.com/apache/sling-org-apache-sling-models-impl/pull/20#issuecomment-702289460
bq. I was thinking that resources can also be wrapped and potentially other objects. True, there is a `ResourceWrapper` but I at least could not say whether it would be expected that for a wrapped resource if the cache should be keyed on the wrapper or the base. I'm also not sure that needs to be dealt with in this PR -- adding support for unwrapping resources could be dealt with at a later time should the desired behavior be clearly defined. ---------------------------------------------------------------- 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]
