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

Unable to schedule SQS Batch Size of greater than 10 #1117

Closed
brucebeck95 opened this issue Mar 18, 2022 · 1 comment
Closed

Unable to schedule SQS Batch Size of greater than 10 #1117

brucebeck95 opened this issue Mar 18, 2022 · 1 comment

Comments

@brucebeck95
Copy link
Contributor

Context

Attempting to create an SQS trigger, through Zappa, that takes in 100 messages at a time.

Expected Behavior

Should be able to add "batch_window" param to zappa_setting.json event and have it schedule.
Should schedule the trigger on SQS successfully.

Actual Behavior

botocore.errorfactory.InvalidParameterValueException: An error occurred (InvalidParameterValueException) when calling the CreateEventSourceMapping operation: Maximum batch window in seconds must be greater than 0 if maximum batch size is greater than 10

Possible Fix

Allow zappa to take in a batch_window argument.

Steps to Reproduce

Add to zappa_settings.json

"function": "function_name",
"event_source": {
"arn": "arn:aws:sqs:eu-central-1:180611847606:queue-name",
"batch_size": 100,
"functionResponseType": "ReportBatchItemFailures",
"enabled": true
}

Your Environment

  • Zappa version used: 0.54.1
  • Operating System and Python version: WSL2 20.04 and Python3.8
monkut added a commit that referenced this issue Nov 19, 2022
monkut added a commit that referenced this issue Dec 1, 2022
* 📝 CHANGELOG.md update for 0.55.1 release

* 📝 add merged PR (#1177) related issues to CHANGELOG.

* 📝 add #1117 to list after merge.

* 📝 adding resolved issue to CHANGELOG

* 🔧 0.55.1 -> 0.56.0
📝 add merged issue to changelog.

* 📝 add #1190

* 🔧 `Pipfile` (pipenv) now used in place of requirements.txt, test_requirements.txt, requirements.in test_requirements.in. Replacing in MANIFEST.in.

Related Issue:
#1199

* 🔧 Add resolved issue #1199
@monkut
Copy link
Collaborator

monkut commented Dec 1, 2022

released in 0.56.0, closing.

@monkut monkut closed this as completed Dec 1, 2022
Ian288 pushed a commit to tackle-io/Zappa that referenced this issue Jul 11, 2023
* 📝 CHANGELOG.md update for 0.55.1 release

* 📝 add merged PR (zappa#1177) related issues to CHANGELOG.

* 📝 add zappa#1117 to list after merge.

* 📝 adding resolved issue to CHANGELOG

* 🔧 0.55.1 -> 0.56.0
📝 add merged issue to changelog.

* 📝 add zappa#1190

* 🔧 `Pipfile` (pipenv) now used in place of requirements.txt, test_requirements.txt, requirements.in test_requirements.in. Replacing in MANIFEST.in.

Related Issue:
zappa#1199

* 🔧 Add resolved issue zappa#1199
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants