I like the overall general approach, however I don't like that the code flow here does not make it clear that `updateCacheConfig` and `updateConfigForGroup` are mutually exclusive.
[ Full content available at: https://github.com/apache/geode/pull/2854 ] This message was relayed via gitbox.apache.org for [email protected]
