Add batched telemetry - Githubissues

microsoft / service-fabric-cli

Service Fabric CLI Tools

Other

53 stars 52 forks source link

Add batched telemetry #166

Closed Christina-Kang closed 5 years ago

Christina-Kang commented 5 years ago

Improve upon #152 to make telemetry send batched telemetry in order to improve performance.

Telemetry will write to a file and hold on to telemetry to send in chunks. Current value is 50 items per batch. There are limited retries, so that if there is something such as no internet connection, we retry intermittently.

Verified:

[ ] Build and test CI passes
[ ] History and readme updated to reflect changes
[ ] Package version updated according to semantic versioning rules
[ ] Tests modified or added, when applicable
[ ] Updated code owners file, when applicable
[ ] Read the PR checklist

coveralls commented 5 years ago

Coverage decreased (-0.9%) to 83.829% when pulling baf248d76dac36d13fa1a9fedd7f9dab0c577edb on Christina-Kang:telemetry-batched into 950b94a507831a27f2c2ea6c5a5bd27a38de3b1b on Microsoft:master.