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

Tables should automatically scale up if below their minimums #202

Closed
GregorStocks opened this Issue Sep 24, 2014 · 5 comments

Comments

Projects
None yet
2 participants
@GregorStocks

GregorStocks commented Sep 24, 2014

I set up my tables and then realized that one of them would need a lot more reads, so I added a min_provisioned_reads for that table. I expected it to automatically scale up, but it didn't. Based on the code, I think min_provisioned_reads is only consulted if we've already decided to scale the table. This surprised me.

@sebdah sebdah added the type: bug label Sep 24, 2014

@sebdah sebdah added this to the 1.17.x milestone Sep 24, 2014

@sebdah sebdah self-assigned this Sep 24, 2014

@sebdah

This comment has been minimized.

Owner

sebdah commented Sep 24, 2014

Thanks for the report.

It seems like a bug (and I come to the same conclusion as you when looking in the code). I'll dig into it.

@sebdah sebdah modified the milestones: 1.18.x, 1.17.x Sep 24, 2014

@sebdah

This comment has been minimized.

Owner

sebdah commented Sep 25, 2014

This has now been fixed and released in version 1.18.4. Please give it a try.

Thanks again for the bug report

@sebdah sebdah closed this Sep 25, 2014

@GregorStocks

This comment has been minimized.

GregorStocks commented Sep 25, 2014

Fix confirmed, thanks!

@GregorStocks

This comment has been minimized.

GregorStocks commented Sep 30, 2014

It looks like this patch doesn't respect Dynamo's "you can only increase the provisioning by 100% at a time" restriction. I have a table that it's trying to adjust from a throughput of 3 directly to 10 (which is the minimum I set), but it fails every time because Dynamo will only let it be adjusted up to 6.

@sebdah

This comment has been minimized.

Owner

sebdah commented Oct 3, 2014

You're right, I'll look into that. Thanks for the report!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment