-
Notifications
You must be signed in to change notification settings - Fork 26
Dynamic queue name for all brokers #100
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
Dynamic queue name for all brokers #100
Conversation
|
@s3rius any chance you can review it (and make a release) soon ? we are missing this feature to build a proper priority queue system |
|
@FlorianLB Hi! I can, s3rius on the vacation now. Could you please check tests? They failed. |
|
@FlorianLB, and please add some information about your PR in the repo's README. |
6341cf2 to
fe2bfdf
Compare
|
@chandr-andr I documented the changes. But tests are failing because of the test pipeline setup, not due to the changes |
|
@FlorianLB Ok, thanks! I'll make a release now |
|
@FlorianLB https://pypi.org/project/taskiq-redis/1.1.1/ |
|
@FlorianLB could you please explain how to use it for low priority queue? E.g. how to configure a worker to consider that one queue has bigger priority than the other? |
You can't right now. You have two options with the current state of the library :
|
|
Thank you for the information! |
Complement to #94 , some brokers were missing the feature.