[KNOX-2144] Alias API KnoxShell support should provide response types better than raw JSON

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.3.0
• Component/s: KnoxShell
• Assignee: Philip Zampino
• Reporter: Philip Zampino
• Created: 04/Dec/19 15:35
• Updated: 06/Dec/19 16:07
• Resolved: 06/Dec/19 16:07

Description

Like many of the other KnoxShell classes, the responses are essentially JSON, which the client has to process itself. For the Alias API interactions, the responses could be processed by the KnoxShell classes themselves, especially since it's a Knox API.

So, instead of 

response = Alias.list(session, clusterName).now()
json = (new JsonSlurper()).parseText( response.string )
cluster = json.topology
aliases = json.aliases

the response type would provide methods for getting the response details

response = Alias.list(session, clusterName).now()
cluster = response.getCluster()
aliases = response.getAliases()

Issue Links

• links to: GitHub Pull Request #211