Two things here: * These methods reduce code size, but I think they make the code less readable as I don't know what they do from the method name / signature. * `CacheFactory.getAnyInstance()` is something we'd like to deprecate, so I prefer the pattern of storing the value in a static (or using `ClusterStartupRule` to handle lifecycle) as it manages the reference and lifecycle in a way that doesn't necessitate globals.
Neither of these is a forcing issue for me; I leave it to your conscience. [ Full content available at: https://github.com/apache/geode/pull/3036 ] This message was relayed via gitbox.apache.org for [email protected]
