Skip to content
This repository was archived by the owner on Oct 24, 2024. It is now read-only.

✨ Add a ttl option for auto expiry of deleted messages #22

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

david-martin
Copy link
Contributor

This change allows a ttl to be set for automatically removing deleted
queue messages. This can be useful for limiting the amount of space used
by queues, while still maintaining some level of auditing.

A new ttl option can be specified, in seconds, for how long before
expiring a deleted message. The default is to not have any expiration
i.e. the existing behaviour.

Some extra assertions are added to the existing indexes test.
A new test for the ttl index is also added.

This change allows a ttl to be set for automatically removing deleted
queue messages. This can be useful for limiting the amount of space used
by queues, while still maintaining some level of auditing.

A new `ttl` option can be specified, in seconds, for how long before
expiring a deleted message. The default is to not have any expiration
i.e. the existing behaviour.

Some extra assertions are added to the existing indexes test.
A new test for the ttl index is also added.
@david-martin
Copy link
Contributor Author

@chilts What are your thoughts on adding this functionality? To date, I've been adding this index separately outside the module. I think it might be useful as a core function of the library.

@davidffrench
Copy link

@chilts Do you have an update on if this should be merged?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants