@adriancole There is no public cache in the most recent patch and the cache is 
hidden behind Reflection2.invokableParamsCache() and therefore external 
visibility should not be an issue. Note that there are 8 existing ```private 
static``` LoadingCaches defined in Reflection2.java that aren't bound by Guice 
either that are shared across all callers. Assuming this pattern that worked so 
far, my patch just follows that. Does that sound good? 

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/164#issuecomment-25639614

Reply via email to