[GEODE-6869] decouple ClusterManagementService query and response types

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Won't Fix
• Fix Version/s: 1.10.0
• Component/s: management
• Assignee: Owen Nichols
• Reporter: Owen Nichols
• Created: 15/Jun/19 03:38
• Updated: 26/Sep/19 18:05
• Resolved: 21/Jun/19 16:17

Description

currently CMS calls like `list(CacheElement)` return CacheElement (or sometimes RuntimeCacheElement which uses inheritance+copy constructor rather than composition to add additional stats alongside the static config info)

this has been "ok" so far, since most CMS calls focus on CRUD operations on things in cache.xml which already extend CacheElement.  However, it would be cleaner to decouple the return type from the filter class, and not require either one to extend CacheElement, to give us more flexibility to expose things via CMS that aren't strictly elements of cache.xml

 

Issue Links

• links to: GitHub Pull Request #3714