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

Typo in docs/reference/async.md #240

Open
aboyett opened this issue Oct 7, 2020 · 1 comment
Open

Typo in docs/reference/async.md #240

aboyett opened this issue Oct 7, 2020 · 1 comment

Comments

@aboyett
Copy link

aboyett commented Oct 7, 2020

The Parallelism section of the Async documentation currently contains "Should you wish to restrict concurrenty for certain functions". Concurrenty is not a word.

Proposed change

The word concurrenty should be changed to concurrency.

-> Alternatively you can increase the parallelism by setting the queue worker's "max_inflight" option to a value greater than one. This will cause the queue-worker to concurrently receive up to max_inflight many messages and simultaneously invoke their corresponding functions. Should you wish to restrict concurrenty for certain functions, please make use of  [multiple queues](#Multiple-queues) and separate these functions accordingly. When scaling up a queue worker, please be aware that you will get up to 'n * max_inflight' parallel function invocations.
+> Alternatively you can increase the parallelism by setting the queue worker's "max_inflight" option to a value greater than one. This will cause the queue-worker to concurrently receive up to max_inflight many messages and simultaneously invoke their corresponding functions. Should you wish to restrict concurrency for certain functions, please make use of  [multiple queues](#Multiple-queues) and separate these functions accordingly. When scaling up a queue worker, please be aware that you will get up to 'n * max_inflight' parallel function invocations.

Context

Your Environment

  • FaaS-CLI version ( Full output from: faas-cli version ):
    N/A

  • Docker version docker version (e.g. Docker 17.0.05 ):
    N/A

  • Are you using Docker Swarm or Kubernetes (FaaS-netes)?
    N/A

  • Operating System and version (e.g. Linux, Windows, MacOS):
    N/A (Linux Fedora 31)

  • Link to your project or a code example to reproduce issue:
    N/A

  • Please also follow the troubleshooting guide and paste in any other diagnostic information you have:

@alexellis
Copy link
Member

Thanks for spotting that. @viveksyngh could you take this please?

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

2 participants