[GitHub] [geode] jinmeiliao commented on pull request #3083: GEODE-6280: Create new public module for the Management API

2019-01-17 Thread GitHub
I see the need to move this functionality out of here. We can probably completely get rid of this method, and have an instantiator to instantiate the instance. e.g. ``` mutator.setCacheLoader(ConfigInstantiator.instantiate(config, cache)); ``` [ Full content available at:

[GitHub] [geode] jinmeiliao commented on pull request #3083: GEODE-6280: Create new public module for the Management API

2019-01-17 Thread GitHub
should this be T instead? [ Full content available at: https://github.com/apache/geode/pull/3083 ] This message was relayed via gitbox.apache.org for notifications@geode.apache.org