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.

Sign up for GitHub

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

Jump to bottom

GPL-766 Convert the Broker class to use a Context Manager workflow #204

Closed
1 task
Chris-Friend opened this issue Dec 3, 2020 · 1 comment
Closed
1 task

GPL-766 Convert the Broker class to use a Context Manager workflow #204

Chris-Friend opened this issue Dec 3, 2020 · 1 comment
Labels
Enhancement New feature or request

Comments

@Chris-Friend
Copy link
Contributor

Chris-Friend commented Dec 3, 2020 •
edited
Loading

User story
As developers, we'd like to change the Broker class to use a context manager workflow. This will simplify calling the class/publishing RabbitMQ messages/error recovery

Who are the primary contacts for this story
Phil
Chris F

Acceptance criteria
To be considered successful the solution must allow:

  • Update the Broker class/RabbitMQ message publish workflow to use a context manager workflow

Dependencies
This story originates from the review of changes required for #203

Additional context
See pika documentation pika/pika#627
@Chris-Friend Chris-Friend added the Enhancement New feature or request label Dec 3, 2020
@Chris-Friend Chris-Friend mentioned this issue Dec 3, 2020
Merged
@pjvv
Copy link
Contributor

pjvv commented Dec 3, 2020

pika/pika#736

@rl15 rl15 changed the title GPL-nnn Convert the Broker class to use a Context Manager workflow GPL-766 Convert the Broker class to use a Context Manager workflow Dec 3, 2020
@rl15 rl15 closed this as completed Dec 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@rl15 @pjvv @Chris-Friend