Skip to content

Conversation

@mergify
Copy link

@mergify mergify bot commented Jul 30, 2024

Similar to other RabbitMQ internal credit flow configurations such as credit_flow_default_credit and msg_store_credit_disc_bound, this commit makes the classic_queue_consumer_unsent_message_limit configurable via advanced.config.

This PR is an alternative to #11822


This is an automatic backport of pull request #11857 done by Mergify.
This is an automatic backport of pull request #11859 done by Mergify.

Similar to other RabbitMQ internal credit flow configurations such as
`credit_flow_default_credit` and `msg_store_credit_disc_bound`, this
commit makes the `classic_queue_consumer_unsent_message_limit`
configurable via `advanced.config`.

See #11822 for the
original motivation to make this setting configurable.

(cherry picked from commit c771b24)
(cherry picked from commit eee851b)

# Conflicts:
#	deps/rabbit/src/rabbit_queue_consumers.erl
@mergify
Copy link
Author

mergify bot commented Jul 30, 2024

Cherry-pick of eee851b has failed:

On branch mergify/bp/v3.13.x/pr-11859
Your branch is up to date with 'origin/v3.13.x'.

You are currently cherry-picking commit eee851bd90.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   deps/rabbit/src/rabbit_queue_consumers.erl

no changes added to commit (use "git add" and/or "git commit -a")

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@mergify mergify bot added the conflicts label Jul 30, 2024
@ansd ansd merged commit ce37242 into v3.13.x Jul 30, 2024
@ansd ansd deleted the mergify/bp/v3.13.x/pr-11859 branch July 30, 2024 09:57
@michaelklishin michaelklishin added this to the 3.13.7 milestone Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants