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

Batch max size question #39

Open
Amertz08 opened this issue Jul 2, 2020 · 4 comments
Open

Batch max size question #39

Amertz08 opened this issue Jul 2, 2020 · 4 comments

Comments

@Amertz08
Copy link

Amertz08 commented Jul 2, 2020

We are attempting to reduce the number of messages being processed by each lambda invocation. We have batching enabled but it doesn't appear that you can configure the message count. It looks like there is a hard coded value for the bytes size of the batch here

private final static int maxBatchSizeBytes = (6 * 1024 * 1024) - 1;
.

Is it even possible?

@dylanmei
Copy link
Contributor

dylanmei commented Jul 2, 2020

6MB is the AWS Lambda payload max limit: https://docs.aws.amazon.com/lambda/latest/dg/gettingstarted-limits.html

Setting a smaller value through configuration, say aws.lambda.batch.max.bytes, seems like a reasonable addition.

@Amertz08
Copy link
Author

Amertz08 commented Jul 2, 2020

Gotcha. What we're wondering is how to only send N messages to a lambda at a time. We are seeing like 7500 messages sent and the lambda is timing out. We'd like to only send a few at a time. This doesn't appear possible with this connector.

@dinegri
Copy link

dinegri commented May 3, 2022

You can control consumer batch size thought consumer.override.max.poll.records in the connector config and enable aws.lambda.batch.enabled, but before that you have to enable override policy in the worker config using connector.client.config.override.policy=All

@ross-paypay
Copy link

ross-paypay commented Mar 13, 2023

For ASYNC invocations, there is a different size limitation in AWS.

I am seeing issues when changing from SYNC to ASYNC mode.

image

I think that the default size should change based on which one of these is set

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