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

Mistake in guest post #29

Closed
StarpTech opened this issue Jan 4, 2020 · 7 comments
Closed

Mistake in guest post #29

StarpTech opened this issue Jan 4, 2020 · 7 comments

Comments

@StarpTech
Copy link

The queue subscriber enables multiple clients to subscribe to the same stream with the same “queue name” and messages will be distributed to each member of the queue group.
https://nats.io/blog/use-cases-for-persistent-logs-with-nats-streaming/

This is not correct. Queue groups are designed to load balance traffic.

@ColinSullivan1
Copy link
Member

ColinSullivan1 commented Jan 6, 2020

@StarpTech , thank you for raising this. This was a guest post, and we don't like altering posts without the author's permission. I do think this can be read accurately - but do admit there's some ambiguity without the first sentence. Messages are distributed among members of the group. @bruth, wdyt?

@bruth
Copy link
Member

bruth commented Jan 6, 2020

That does read a bit odd now that you bring it up. This is what the NATS docs states for queue groups:

NATS provides a built-in load balancing feature called distributed queues. Using queue subscribers will balance message delivery across a group of subscribers which can be used to provide application fault tolerance and scale workload processing.

Maybe it should say "messages will be load balanced across members of the queue group"?

@StarpTech
Copy link
Author

StarpTech commented Jan 6, 2020

@bruth The whole sentence makes sense to me because of the keyword load balancing but in the guest post, there is no reference about it.

@bruth
Copy link
Member

bruth commented Jan 6, 2020

@StarpTech Sure, I am suggesting that we change what I wrote to this:

The queue subscriber enables multiple clients to subscribe to the same stream with the same “queue name” and messages will be load balanced across members of the queue group.

@bruth
Copy link
Member

bruth commented Jan 6, 2020

@ColinSullivan1 You have my permission to change the text to the above. 😉

@gcolliso
Copy link
Member

gcolliso commented Jan 6, 2020

Thanks @ColinSullivan1 , @bruth and @StarpTech I will make the update.

@gcolliso
Copy link
Member

gcolliso commented Jan 6, 2020

fixed with PR#532 nats-io/nats-site#532

@gcolliso gcolliso closed this as completed Jan 6, 2020
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

No branches or pull requests

4 participants