Skip to content

NXCALS-4012 Restore default, lazy load of VariableConfigs for VariableData

Grigorios Avgitidis requested to merge fix-NXCALS-4012-assign_variable_to_hierarchy_time into develop

Merge request template

COMPATIBILITY NOTE

Please make sure that this change is BACKWARDS COMPATIBLE and a new server will work correctly with clients using old APIs versions.

It may be that data sent in the new format will break the old clients.

In case of any potential incompatibility we have to inform our clients and schedule release & deployment with them.

Tested on env DEV_GAVGITID

The bellow test is simulating conditions similar with provided use-case by the author of the related issue. More information about the use-case and query can be found on the linked ticket.

Before

2020-08-04 16:21:07.169 [DEBUG] [main] HierarchyClient - [HierarchyClient#addVariables] ---> POST https://nxcals-service/hierarchies/10000126/variables-ids HTTP/1.1 2020-08-04 16:21:08.756 [DEBUG] [main] HierarchyClient - [HierarchyClient#addVariables] <--- HTTP/1.1 200 (1585ms)

Duration millis: 1618

After

2020-08-04 17:12:26.801 [DEBUG] [main] HierarchyClient - [HierarchyClient#addVariables] ---> POST https://nxcals-service/hierarchies/10000126/variables-ids HTTP/1.1 2020-08-04 17:12:27.507 [DEBUG] [main] HierarchyClient - [HierarchyClient#addVariables] <--- HTTP/1.1 200 (705ms)

Duration millis: 748

@acc-logging-team - please have a look for approval.

Please describe the merge request here.

Closes NXCALS-4012

Merge request reports