SI is too chatty with SC

[distantcam]

ServiceInsight will often send 2 or 3 web requests to SC for every action that it does. This is fine when SC is localhost but not so good when running over the internet.

Instead, SI should cache web requests, which should help with overall performance.

[dannycohen]

@distantcam - the issue is phrased too generally and it is unclear what needs to be done / fixed.

• Which actions are chatty ?
• How chatty are these actions ?
• What is the chattiness interaction delay over the internet (estimated) ?

[distantcam]

@dannycohen I was still investigating, and made this issue as a reminder to come back to this.

I was noticing repeated calls to the same url from SI in the logs. It turns out this is when a different message from the same saga is selected, and SI isn't smart enough to realize it's the same data and avoid the call to SC.

Given saga data can be up to 3mb each, avoiding having to process that much data everytime would be nice.

[dannycohen]

@distantcam - Makes sense. please ping when you get to analyze this in depth.

when a different message from the same saga is selected, and SI isn't smart enough to realize it's the same data and avoid the call to SC

Note that in such a case, caching only would be bad, since the Saga data may have changed. Checking for changes in the data on the server side would be a good option.

[johnsimons]

Fixes as part of #455