Skip to content

AMQP: Use exchange instead of queue for discovery (WIP) #27

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
Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

AMQP: Use exchange instead of queue for discovery (WIP) #27

wants to merge 3 commits into from

Conversation

jonnor
Copy link
Member

@jonnor jonnor commented Feb 27, 2017

No description provided.

@jonnor
broker.subscribeParticipantChange(): Add callback
ca636ff
@jonnor
tests: Check discovery between client/broker
6a87305 
Was completely uncovered in our tests.
Though `msgflo` has some that probably exercise this
@jonnor jonnor force-pushed the amqp-discovery-exchange branch 2 times, most recently from 513a709 to a1db2bc Compare February 27, 2017 17:37
@jonnor jonnor force-pushed the amqp-discovery-exchange branch from a1db2bc to 047e7f6 Compare February 27, 2017 17:52
@jonnor jonnor mentioned this pull request Feb 27, 2017
Open
5 tasks
@jonnor
Copy link
Member Author

jonnor commented Feb 27, 2017

Keeping this open until

  1. It is clear whether sending to fbp exchange from participant, when a fbp queue already exists will automatically put the message into the fbp queue.
  2. There are pull requests open for the participant libraries.

Need to bump version after this change also.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jonnor