Skip to content

Integrate TransmissionStateManager and TransmitFromStorageHandler with transmitter#35158

Merged
vishweshbankwar merged 1 commit intomainfrom
vibankwa/updated-transmission-from-storage
Mar 27, 2023
Merged

Integrate TransmissionStateManager and TransmitFromStorageHandler with transmitter#35158
vishweshbankwar merged 1 commit intomainfrom
vibankwa/updated-transmission-from-storage

Conversation

@vishweshbankwar
Copy link
Copy Markdown
Contributor

@vishweshbankwar vishweshbankwar commented Mar 27, 2023

This is the final PR, it integrates the changes done in in #35017 and #34926 with AzureMonitorTransmitter.

Summary: With this change, transmission from storage will be handled by TransmitFromStorageHandler. It will periodically check for any files available in the storage for retries. In addition to this, TransmissionStateManager will manage the transmission state by implementing exponential back-off in case of transient errors.

@vishweshbankwar vishweshbankwar changed the title Updated transmission from storage Integrate TransmissionStateManager and TransmitFromStorageManager with transmitter Mar 27, 2023
@vishweshbankwar vishweshbankwar changed the title Integrate TransmissionStateManager and TransmitFromStorageManager with transmitter Integrate TransmissionStateManager and TransmitFromStorageHandler with transmitter Mar 27, 2023
@vishweshbankwar vishweshbankwar marked this pull request as ready for review March 27, 2023 16:42
@azure-sdk
Copy link
Copy Markdown
Collaborator

API change check

API changes are not detected in this pull request.

Copy link
Copy Markdown
Member

@rajkumar-rangaraj rajkumar-rangaraj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vishweshbankwar vishweshbankwar merged commit e72578a into main Mar 27, 2023
@vishweshbankwar vishweshbankwar deleted the vibankwa/updated-transmission-from-storage branch March 27, 2023 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants