Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add option to make the telemetry queue persistent #154

Merged

Conversation

Projects
None yet
2 participants
@c-w
Copy link
Member

commented Mar 6, 2019

Currently the Application Insights client SDK already has some great resiliency built-in, e.g. retrying to send telemetry if there are intermittent connectivity issues or service interruptions. However,
there still is a risk to lose telemetry if the app crashes before sending telemetry since the telemetry queue is in-memory only.

This change adds an option to persist the queue to disk via the persist-queue module which will protect customers against potential data loss, at the cost of a slight loss in performance.

@c-w c-w force-pushed the CatalystCode:add-queue-persistence-option branch 4 times, most recently from b2ade2f to f5d3351 Mar 8, 2019

Add option to make the telemetry queue persistent
Currently the Application Insights client SDK already has some great
resiliency built-in, e.g. retrying to send telemetry if there are
intermittent connectivity issues or service interruptions. However,
there still is a risk to lose telemetry if the app crashes before
sending telemetry since the telemetry queue is in-memory only.

This change adds an option to persist the queue to disk via the
[persist-queue](https://pypi.org/project/persist-queue/) module which
will protect customers against potential data loss, at the cost of a
slight loss in performance.

@c-w c-w force-pushed the CatalystCode:add-queue-persistence-option branch from f5d3351 to fcebaa2 Mar 8, 2019

@SergeyKanzhelev SergeyKanzhelev merged commit b71802a into microsoft:develop Mar 18, 2019

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
license/cla All CLA requirements met.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.