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

Old logs and 24h batching limit #29

Closed
alexeykaplin opened this issue Nov 25, 2019 · 2 comments
Closed

Old logs and 24h batching limit #29

alexeykaplin opened this issue Nov 25, 2019 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@alexeykaplin
Copy link

alexeykaplin commented Nov 25, 2019

I am getting lots of errors on an initial log sync:
time="2019-11-25T11:46:49Z" level=error msg="[cloudwatch 0] InvalidParameterException: The batch of log events in a single PutLogEvents request cannot span more than 24 hours.\n\tstatus code: 400, request id: 9e371dc0-5efe-4803-91d9-e779cb6bc165\n"

Possible solution is an appropriate duration limiter, it can be introduced here:

if len(stream.logEvents) == maximumLogEventsPerPut || (stream.currentByteLength+cloudwatchLen(event)) >= maximumBytesPerPut {

@alexeykaplin alexeykaplin changed the title Old logs and 24h limit Old logs and 24h batching limit Nov 25, 2019
@PettitWesley PettitWesley self-assigned this Dec 3, 2019
@PettitWesley PettitWesley added the enhancement New feature or request label Dec 3, 2019
@PettitWesley
Copy link
Contributor

@alexeykaplin We appreciate contributions! Especially for small things like this.

@alexeykaplin
Copy link
Author

alexeykaplin commented Dec 3, 2019

PR can be found there: #32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants