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

feature request: ability to issue jobs to queue with included Time To Live duration #2

Closed
c-nv-s opened this issue Oct 18, 2023 · 4 comments · Fixed by #3
Closed

feature request: ability to issue jobs to queue with included Time To Live duration #2

c-nv-s opened this issue Oct 18, 2023 · 4 comments · Fixed by #3

Comments

@c-nv-s
Copy link

c-nv-s commented Oct 18, 2023

I was searching for a queue backed by sqlite that could offer this TTL/Deadline feature that is available in another Postgres-backed queue that I am also using (I'll include the project link below). I've copied the following issue request verbatim from that project to see if you could also consider it for this project because the feature is so very useful:

In some use cases it is very handy to be able to automatically remove a job from the queue if it has not been worked by a specified time duration.

This request is to implement an option/mechanism which allows the developer to specify a duration or Time To Live (TTL) for the job/s issued to the queue.

Personally, I'm not too precious about whether the developer is allowed to set the option at client creation or per request, but I understand that others may be more opinionated about it.

@c-nv-s
Copy link
Author

c-nv-s commented Oct 18, 2023

implementation of the feature on the postgres-backed queue for reference:
https://github.com/acaloiaro/neoq/pull/59/files

@risico
Copy link
Owner

risico commented Nov 7, 2023

That's a good idea! I'll try and add it today.

(sorry for the late reply, I just saw the issue)

@c-nv-s
Copy link
Author

c-nv-s commented Jan 2, 2024

@risico did you manage to make any progress with this feature?

@risico
Copy link
Owner

risico commented Jan 17, 2024

Sorry I just noticed the message, for some reason Github is not sending me the notifications. But yeah, I've just pushed the final code for this: #3

Will merge to master in a bit.

@risico risico closed this as completed in #3 Jan 17, 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 a pull request may close this issue.

2 participants