Okay, my changes do not work as expected. For whatever reason messages from delayed quorum queues vanish occasionally.

I'll have to investigate why later.

Hey!

I think @ruudk has recently worked with this code. Maybe they can help review this?

Cheers!

Carsonbot

Alright, I've made some changes, it works now. Please let me know if you need a DOC PR or if this should go to 6.3 instead of 6.2.

Hi @fwolfsjaeger, I add the same issue and went almost with the same strategy as you.

However, if I understand correctly your PR, I have to know in advance the routingKeys (the name of the delay queues, that depends of the name of the original queue and the delays).
And in my use case, it is something that is unknown by the dev team and is handled by the "infra guys".

I thought is was easier to only precise the extra args for the associated delay queues.

That is why I created a PR that achieve the same goal.
I do not mean to compete your PR.

Hi @fwolfsjaeger, I add the same issue and went almost with the same strategy as you.

However, if I understand correctly your PR, I have to know in advance the routingKeys (the name of the delay queues, that depends of the name of the original queue and the delays). And in my use case, it is something that is unknown by the dev team and is handled by the "infra guys".

I thought is was easier to only precise the extra args for the associated delay queues.

That is why I created a PR that achieve the same goal. I do not mean to compete your PR.

Hey!

You are right, with my PR, you have to know the actual queue name. I have added the config array arguments_delay to the queue options. This way it is possible to change settings per queue.

In my case a transport wide setting would be sufficient as well though. I don't know if anybody actually requires this config on queue level. Your PR as based on 6.3 and includes a test case, I'll close mine.