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

Jump to bottom

Topic documentation on our publish-subscribe architecture. #1547

Merged
merged 1 commit into from Dec 2, 2019
Merged

Topic documentation on our publish-subscribe architecture. #1547

merged 1 commit into from Dec 2, 2019

Conversation

Reviewers

@tlyu tlyu

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@nmathewson @coveralls @tlyu @torproject-pusher
@nmathewson
Copy link
Contributor

@nmathewson nmathewson commented Nov 16, 2019

No description provided.

@nmathewson
Topic documentation on our publish-subscribe architecture.
d700dc7
@coveralls
Copy link

@coveralls coveralls commented Nov 16, 2019

Pull Request Test Coverage Report for Build 7264

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 62.426%
Totals Coverage Status
Change from base Build 7256: 0.0%
Covered Lines: 48723
Relevant Lines: 78049
💛 - Coveralls

tlyu
tlyu reviewed Nov 21, 2019
View changes
src/lib/pubsub/publish_subscribe.md
number of message types. Where channels vary is their delivery
mechanisms, and their guarantees about when messages are processed.

Currently, three delivery policies are possible:
Copy link
Contributor

@tlyu tlyu Nov 21, 2019

Maybe link to the mainloop_pubsub.h definitions?

@torproject-pusher torproject-pusher merged commit d700dc7 into torproject:master Dec 2, 2019
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment