Added config option to allow disabling x-expires in getDelayQueueArguments
#627
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I am using quorum and i noticed that the messages are deleted before they're moved to their destination queue.
By disabling the
x-expires, my messages are no longer lost.I think is considerate to have the ability to enable/disable delay queue expires, to avoid issues for users that use quorum.
Found other issue related -> #601
From the TTL documentation (/ttl):
This page describes the key difference for x-expires.
From the Quorum Queues documentation (/quorum-queues):
This page describes the core design that explains why the TTL behavior is different.
When combined, these two pages explain that the "unused" criteria for Classic queues (which prevents deletion if messages are present) does not apply to the more robust, data-safe Quorum queues, which prioritize consistency and guaranteed cleanup.