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

SQS has a maximum of 10 messages #31614

Closed
wants to merge 1 commit into from
Closed

SQS has a maximum of 10 messages #31614

wants to merge 1 commit into from

Conversation

ugosan
Copy link
Contributor

@ugosan ugosan commented May 12, 2022

Worth mentioning SQS allows a maximum of 10 messages in a batch, as per their doc:
https://docs.aws.amazon.com/AWSSimpleQueueService/latest/SQSDeveloperGuide/quotas-messages.html

Worth mentioning SQS allows a maximum of 10 messages in a batch, as per their doc:
https://docs.aws.amazon.com/AWSSimpleQueueService/latest/SQSDeveloperGuide/quotas-messages.html
@ugosan ugosan added docs Team:Docs Label for the Observability docs team labels May 12, 2022
@elasticmachine
Copy link
Collaborator

Pinging @elastic/obs-docs (Team:Docs)

@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels May 12, 2022
@mergify mergify bot assigned ugosan May 12, 2022
@ugosan ugosan added Team:Docs Label for the Observability docs team and removed Team:Docs Label for the Observability docs team labels May 12, 2022
@ugosan ugosan requested a review from andrewkroh May 12, 2022 15:06
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-05-12T14:58:50.526+0000

  • Duration: 20 min 47 sec

❕ Flaky test report

No test was executed to be analysed.

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

  • /package : Generate the packages and run the E2E tests.

  • /beats-tester : Run the installation tests with beats-tester.

  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@@ -212,8 +212,8 @@ The default is `10 MiB`.
[float]
==== `max_number_of_messages`

The maximum number of SQS messages that can be inflight at any time. Defaults
to 5.
The maximum number of SQS messages that can be inflight at any time, with a maximum of 10. Defaults
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this in response to some error observed while using the input?

If so then I think there may be an issue in the code. I intended to allow max_number_of_messages > 10 while only calling ReceiveMessage with a maximum value of 10. So the input would attempt to maximum the usages of the workers while splitting up the calls to ReceiveMessage into batches no larger than 10.

@jlind23
Copy link
Collaborator

jlind23 commented Nov 28, 2023

Closing this for now as it is outdated, but feel free to reopen it if needed.

@jlind23 jlind23 closed this Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Team:Docs Label for the Observability docs team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants