Skip to content
This repository has been archived by the owner on Apr 17, 2024. It is now read-only.

Unable to add when date range is too big #20

Open
parberge opened this issue Sep 3, 2019 · 7 comments
Open

Unable to add when date range is too big #20

parberge opened this issue Sep 3, 2019 · 7 comments

Comments

@parberge
Copy link
Member

parberge commented Sep 3, 2019

It seems that we can't create all the events requested when they exceed a certain number. I tried with over 30 days when I ran into this problem

@parberge parberge added the bug Something isn't working label Sep 3, 2019
@parberge
Copy link
Member Author

parberge commented Sep 3, 2019

I can see that the timereport-api respons with a 500 (log taken from timereport-slack):

[DEBUG]	2019-09-03T16:26:45.194Z	3db0ae5d-238f-4b9a-8579-a8922a45d524	"POST /api/event/users/<user_id_removed> HTTP/1.1" 500 2030

@parberge
Copy link
Member Author

parberge commented Sep 3, 2019

This is the reason:

pynamodb.exceptions.PutError: Failed to put item: An error occurred (ProvisionedThroughputExceededException) on request (THTAD2C3FPSS3KTEOGIU6MI4TNVV4KQNSO5AEMVJF66Q9ASUAAJG) on table (dev_event) when calling the PutItem operation: The level of configured provisioned throughput for the table was exceeded. Consider increasing your provisioning level with the UpdateTable API.

Questions:

  • What is the limit, i.e how fast can we write?
  • Should we introduce sleep to prevent to fast writes or increase "capacity"

@parberge parberge added the question Further information is requested label Sep 3, 2019
@parberge
Copy link
Member Author

parberge commented Sep 3, 2019

@tommymalmqvist @kamigerami Any inputs on this issue?

@parberge
Copy link
Member Author

parberge commented Sep 4, 2019

@parberge parberge removed bug Something isn't working question Further information is requested labels Sep 4, 2019
@kamigerami
Copy link
Member

yes - exponential backoff or auto increase limit (they support that in dynamo nowadays)

@parberge
Copy link
Member Author

@mattiaslundberg With the move to using batch, have we tested this with big date ranges?

@mattiaslundberg
Copy link
Contributor

Doesn't look any better :( We likely need to figure out something else... (Tried to add a range of 90 days and only ~25 are saved)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants