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 note about duplicate entries in subscription queue #4286
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you think of this minor improvement @ramonsmits?
persistence/msmq/subscription.md
Outdated
@@ -7,7 +7,9 @@ related: | |||
- samples/msmq/persistence | |||
--- | |||
|
|||
Note: The subscription queue can contain duplicate entries and is by design. This does not result in events to be published multiple times. After a publisher restarts the subscription queue state will be rewritten and deduplicated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: The subscription queue can contain duplicate entries and is by design. This does not result in events to be published multiple times. After a publisher restarts the subscription queue state will be rewritten and deduplicated. | |
Note: The subscription queue can contain duplicate entries and is by design. This does not result in events to be published multiple times as publishers deduplicates at publishing time. Each time a publisher is restarted the subscription queue state will be rewritten and deduplicated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mauroservienti it is incorrect, it doesn't dedupe during publishing, it dedupes during reading the subscriptions and when receiving a subscription request from a subscriber. Let me see if I can rewrite.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm fine with either solution.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's still not really clear to me, but I'm happy with the change anyway.
Based on forum post https://discuss.particular.net/t/msmq-persistence/1014/21