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

Update worker to be under consumer #89

Merged
merged 5 commits into from
Oct 21, 2021
Merged

Update worker to be under consumer #89

merged 5 commits into from
Oct 21, 2021

Conversation

nsweeting
Copy link
Owner

@nsweeting nsweeting commented Oct 21, 2021

Removes workers as being a global entity that all consumers use. In high traffic scenarios - this causes many race conditions and errors on shutdown. Workers are now a child of the consumer.

Since rabbit no longer requires starting a worker supervisor - the use of an application has been removed.

@nsweeting nsweeting merged commit 46db8ac into v0.16 Oct 21, 2021
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.

1 participant