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

Order stored messages #77

Conversation

nedimtokic
Copy link
Collaborator

@nedimtokic nedimtokic commented Jul 19, 2024

When using the persistence mechanism, SDK resends messages to the broker, but not in the order in which they were sent.

Before resending messages from the database, we are waiting for the MQTTNet library to clean its queue, which holds the newest 10k messages sent by user.

This PR drops new messages from the MqttNet library queue to ensure sending stored messages according to how they were streamed by the user. New messages will be sent from the database.

Drop new messages from the MqttNet library queue
to ensure sending stored messages according to
how they were streamed by the user

Signed-off-by: nedimtokic <nedim.tokic@secomind.com>
@nedimtokic nedimtokic force-pushed the astarte-sdk-csharp-message-queue-order branch from c004f4f to 3c65c4f Compare July 19, 2024 12:52
@harlem88 harlem88 requested a review from osmanhadzic July 19, 2024 13:04
@harlem88 harlem88 merged commit d44adf6 into astarte-platform:master Jul 19, 2024
4 of 5 checks passed
@harlem88 harlem88 added this to the v0.7 milestone Nov 12, 2024
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.

3 participants