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

ARTEMIS-5304 Allow delayed close of federation consumers #5493

Merged

Conversation

tabish121
Copy link
Contributor

When demand is removed from a federation consumer and the AMQP link is drained allow the consumer to idle for a configurable time period before closing the link. If demand returns before the idled link is closed we can recover the link by simply granting more credit.

@tabish121 tabish121 force-pushed the amqp-federation-delay-receiver-close branch from 6b082db to 35cd587 Compare February 12, 2025 16:15
@jbertram
Copy link
Contributor

I was expecting some documentation for this new setting, but I couldn't find it. Was that an oversight or does this not need any user-facing docs?

@tabish121
Copy link
Contributor Author

I was expecting some documentation for this new setting, but I couldn't find it. Was that an oversight or does this not need any user-facing docs?

As follow work for this plus many other updates there will need to be rework of the federation docs but that's a bit out of scope here. The defaults on this should suffice for the majority of users as this should be transparent in practice.

@tabish121 tabish121 force-pushed the amqp-federation-delay-receiver-close branch from 35cd587 to 79ef2f4 Compare February 13, 2025 23:09
When demand is removed from a federation consumer and the AMQP link is
drained allow the consumer to idle for a configurable time period before
closing the link. If demand returns before the idled link is closed we
can recover the link by simply granting more credit.
@tabish121 tabish121 force-pushed the amqp-federation-delay-receiver-close branch from 79ef2f4 to df4658c Compare February 14, 2025 14:33
@tabish121
Copy link
Contributor Author

I've squashed the changes and rebased with main.

@gemmellr gemmellr merged commit 272298f into apache:main Feb 14, 2025
6 checks passed
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 this pull request may close these issues.

3 participants