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

fix/Broadcast-fail-by-give-queue-kwargs #906

Merged
merged 2 commits into from
Aug 15, 2018

Conversation

htkuan
Copy link
Contributor

@htkuan htkuan commented Aug 15, 2018

Because give Broadcast(name='xxx', queue='xxx'),
it will create one specific name for queue,
and if launch more worker for consuming this queue,
this specific name queue already exist then it will not create new queue to
binding fanout exchange.
So all worker will consuming same queue, and exchange only binding this
queue then lose the effect of broadcasting.

Because give Broadcast(name='xxx', queue='xxx'),
it will create one specific name for queue,
and if launch more worker for consuming this queue,
this specific name queue already exist then it will not create new queue to
binding fanout exchange.
So all worker will consuming same queue, and exchange only binding this
queue then lose the effect of broadcasting.
@htkuan htkuan changed the title fix/ fix/Broadcast-fail-by-give-queue-kwargs Aug 15, 2018
@htkuan
Copy link
Contributor Author

htkuan commented Aug 15, 2018

test fail on Broadcast('test_Broadcast', 'explicit_queue_name'),
it want 'explicit_queue_name', but it is reason for broadcast fail...

@htkuan htkuan force-pushed the fix/Broadcast-fail-by-give-queue-kwargs branch from 60cbb10 to 4781382 Compare August 15, 2018 10:01
@htkuan
Copy link
Contributor Author

htkuan commented Aug 15, 2018

Sorry, I don't understand the "AppVeyor build failed" error.
It seems ci service error?

@auvipy
Copy link
Member

auvipy commented Aug 15, 2018

yes.

@auvipy auvipy merged commit 87d6423 into celery:master Aug 15, 2018
@auvipy auvipy added this to the 4.3 milestone Aug 15, 2018
thedrow added a commit that referenced this pull request Mar 27, 2019
thedrow added a commit that referenced this pull request Mar 27, 2019
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