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

Discard message queue items that have timed out #76

Open
mdbergmann opened this issue Jan 16, 2024 · 0 comments
Open

Discard message queue items that have timed out #76

mdbergmann opened this issue Jan 16, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@mdbergmann
Copy link
Owner

mdbergmann commented Jan 16, 2024

Sending messages to an actor with timeout will be processed even if the caller receives a 'ask-timeout' condition.

I think it should be possible to include a universal timestamp to the queue item for when the item is obsolete.
When it is obsolete, processing it can be discarded.

Care must be taken to not discard a queue item that has not timed out yet. So we should give it a few seconds more.

IDEA: we could spare the wheel-timer if we would raise the ask-timeout condition only in the above case and at the queue level.
The caveat would be that when an actor is 'asked' with a timeout of say 20 seconds, but the queue item is handled after 30 seconds, then the user would get the timeout just then, but not at 20 seconds as he specified.

Yet, checking at the time of processing the message would be fine still.

@mdbergmann mdbergmann added the enhancement New feature or request label Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant