-
Notifications
You must be signed in to change notification settings - Fork 17
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
implement IPersistTimeoutsV2 interface #97
Conversation
@mauroservienti @DavidBoike @danielmarbach Could you have a look? |
Actual code looks fine to me. But (I'm sure this is just my own lack of context) why the references to NServiceBus.5.2.8-beta0001? |
@DavidBoike This is because we have dependency on fixes we need to do in core at the same time. We manually uploaded package to myget, but once the actual release is done I'll change it to the correct version. |
d0aa78a
to
80b8ce7
Compare
Makes sense. I'm 👍 (On all the others too, since they're all basically the same.) |
80b8ce7
to
fd2242f
Compare
try | ||
{ | ||
CoreTimeoutData timeoutData; | ||
return TryRemove(timeoutId, out timeoutData); |
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.
Remind me: Why don't we have that try catch for the concurrency ex? (in the original try remove)
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.
Duh, strike that
Looks good! |
implement IPersistTimeoutsV2 interface
Who's affected
You might be affected if you are using one of the following transport configurations:
and you are using timeouts or deferred messages (e.g.
Bus.Defer(...)
)Symptoms
When experiencing connectivity issues with the transport there's a chance that a due timeout/deferred message is lost. Therefore the message will never arrive.
What to do if you are affected
We highly recommend to update to the latest patch versions of your NServiceBus and optional persistence and transport packages. For more details see issue Particular/NServiceBus#2885.
Connects to Particular/NServiceBus#2885