Jira Legacy | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|
|
Table of Contents |
---|
...
For the JSON output of Get DataNode we need to alter the DataMapUtils class. The function toDataMap translates a datanode object to a JSON output. We only want this to occur on the top parent node and as such we can create a new method which is called before toDataMap:
public static Map<String, Object> toDataMapTopNodetoDataMapParentNode(final DataNode dataNode) {
final String nodeName = dataNode.getXpath().substring(dataNode.getXpath().lastIndexOf('/') + 1)
.replaceAll("\\[.*?\\]", "");
return ImmutableMap.<String, Object>builder().put(nodeName,
toDataMap(dataNode)
).build();
}
We only want the container name on the top highest parent level of the JSON output and as such have to distinguish between a top parent level node and non top parent level node. We create an outer map to wrap the inner map which creates the appropriate levels in the JSON output.
...
Queries which use toDataMap:
Effected Components | Effected Clients | Query/Service | Class | Notes | Response now | Response After |
---|
Decision | |||||||
---|---|---|---|---|---|---|---|
CPS Core | CPS TBDMT | Get Data Node (as above) | DataRestController.java | This change is given in the example above. The JSON output for Get Node will be updated. |
| { "bookstore-name":"Chapters", | Don't update version, check with tbdmt about impact of new JSON output for getNode/queryNode. |
CPS Core | CPS Temporal | Notification Service | CpsDataUpdatedEventFactory |
.java | Notification Service response should be the same as CPS Core for consistency. CPS Temporal | cpsDataUpdatedEvent Object content.data.additionalProperties.toString(): [[bookstore-name:Chapters]] | cpsDataUpdatedEvent Object content.data.additionalProperties.toString(): [bookstore:[bookstore-name:Chapters]] |
CPS Temporal only recording and not processing this data. Therefore change (response after) is OK | |||||||
CPS Core | CPS TBDMT | Query data nodes | QueryRestController.java | Query DataNodes will produce a datanode within an array with the container ID and then the relevant data: | [ { "code": "01", | [ "categories": { "code": "01", | Don't update version, check with tbdmt about impact of new JSON output for getNode/queryNode. |
CPS Core | None | get Node by CMHandle and Xpath (NCMP) | NetworkCmProxyController.java | This method is deprecated and is no longer used as part of NCMP. |
Outstanding issues:
Backwards Incompatibility:
Keep version | New version | New endpoints | |
---|---|---|---|
Action | Keep the current version of the interface and alert effected client TBDMT of the changes. | Release version 2 of the interface with changes while retaining v1 endpoints and output. | Create new endpoints and deprecate the old endpoints sometime in the future. Keep the endpoints for get and query which we currently have and release new endpoints that have container context. |
Preference/Impact | Current preferred option as long as TBDMT agree. Impacts TBDMT. | Would need its own spike to analyze the impact. CPS Temporal would need the context of if request is coming from v1 or v2 for notification service. | A lot of duplicated code which will need to be depracated in the future. CPS Temporal would need the context of if request is coming from new or old endpoints. |