Read throughput not increased because write should be decreased.. #133

Closed
doapp-jeremy opened this Issue Mar 24, 2014 · 6 comments

Projects

None yet

2 participants

@doapp-jeremy

I seem to have run into a situation where my read throughput should be increasing, but my writes should be decreasing, but neither is happening.

I have always-decrease-rw-together: true, so that's why the write's aren't decreasing, which is the behavior I want, however, I would expect the reads to be increased.

@sebdah sebdah added this to the 1.10.x milestone Mar 24, 2014
@sebdah sebdah self-assigned this Mar 24, 2014
@sebdah
Owner
sebdah commented Mar 24, 2014

Thanks for the report @doapp-jeremy, I'll dig into it. You don't happen to have a log showing this behaviour (preferably a debug log, but any log is helpful)?

@sebdah
Owner
sebdah commented Mar 24, 2014

Never mind the log, I have it narrowed down. Thanks anyway.

@sebdah sebdah added a commit that referenced this issue Mar 24, 2014
@sebdah Added release notes for #133 20d0ce0
@sebdah
Owner
sebdah commented Mar 24, 2014

I have written a fix for this. It is released in 1.10.2 beta 1. Could you please test and see if that solves the problem for you? It can be installed by running

pip install dynamic-dynamodb==1.10.2b1

Thanks

@doapp-jeremy

I installed 1.10.2b1 and restarted. It seems like it's ignoring the always-decrease-rw-together: true

I would expect that if either read or write needs to be increased, those values would be increased, but if either needs to be decreased, they would only be decreased if both need to be if always-decrease-rw-together is true.

@sebdah sebdah added a commit that referenced this issue Mar 26, 2014
@sebdah Added logging info #133 7e924dc
@sebdah
Owner
sebdah commented Mar 26, 2014

I looked in to this, it seems like it's just a matter of logging issues. The updated version did not proper log when it couldn't scale down due to always-decrease-rw-together. I have released 1.10.2b2 with a fix for that.

It looks good when I ran a quick test now, but I'm travelling and have quite limited testing capabilities.

@sebdah
Owner
sebdah commented Mar 29, 2014

This has now been released in version 1.10.2. Thanks for the bug report!

@sebdah sebdah closed this Mar 29, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment