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

Adding Channel to documentation #3475

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

hkateu
Copy link

@hkateu hkateu commented Sep 14, 2024

Hi,
I added documentation for Channel based on issue2866.

I also think that for new users to easily follow the documentation, the concurrency primitives, we can do the following:

  • Change the order in the docs to: Signal , Channel, Topic then Queue. Going from a Channel to a Topic seems logical since its a many-to-one vs many to many pub-sub relationship.
  • Show that Topics are used for a many-to-many pub-sub scenario, since it's not explicitly mentioned in the docs.

Any feedback is welcome.

testdata/tom_sawyer.txt Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants