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

Several smaller bugs and improvements found while running in production #8

Merged
merged 5 commits into from Sep 12, 2020

Conversation

TrueBrain
Copy link
Member

No description provided.

… empty

If an empty map-name is written, PynamoDB removes the entry from
the dict (which is fine), but upon reading it fails validation as
the field is not there. By allowing nullable map_name, this is
fixed.
In AWS the DynamoDB table names are per region, meaning that if
you run staging and production in the same account, they would use
the same database. this is unwanted of course. Allowing to set a
prefix resolves this issue.
While running the master-server for a while, we gathered enough
data to balance the privisioned RCU/WCU better, resulting in a
more responsive master-server.
As multiple instances start up at nearly the same time, they would
all start their stale-check on nearly the same time. This is both
unwanted and gives stress on the database server.
@TrueBrain TrueBrain merged commit f2bca4a into OpenTTD:master Sep 12, 2020
@TrueBrain TrueBrain deleted the more_prod_bugs branch September 12, 2020 09:13
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

Successfully merging this pull request may close these issues.

None yet

2 participants