Hi Dan, I've added a comment to https://github.com/apache/isis/commit/ad79a943c51473f81b33bc0f84b17fc2819d0ea6#diff-e6d4ba72d6d81e5f75653a968c739d17R176 but I'm not sure that will send a notification to dev@ or even personally to you as the committer of the change.
I don't know what is the scope of ServiceInstantiator but the new caches for @PostConstruct and @PreDestroy look to me as application/global scoped. If this is the case then I think using a concurrent map would be better than LinkedHashMap. Martin Grigorov Wicket Training and Consulting https://twitter.com/mtgrigorov
