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

Dynamodb return all items #24

Merged
merged 1 commit into from
Apr 19, 2019
Merged

Dynamodb return all items #24

merged 1 commit into from
Apr 19, 2019

Conversation

vcastellm
Copy link

Due to:

If the total number of scanned items exceeds the maximum data set size limit of 1 MB, the scan stops and results are returned to the user as a LastEvaluatedKey value to continue the scan in a subsequent operation. The results also include the number of items exceeding the limit. A scan can result in no table data meeting the filter criteria.

Items should be retrieved from pages and decoded.

@vcastellm vcastellm requested a review from abronan April 13, 2019 08:49
@coveralls
Copy link

coveralls commented Apr 13, 2019

Coverage Status

Coverage increased (+0.03%) to 84.471% when pulling e04dc0b on dynamodb_list_all into 8fd1c7c on master.

Due to:

If the total number of scanned items exceeds the maximum data set size limit of 1 MB, the scan stops and results are returned to the user as a LastEvaluatedKey value to continue the scan in a subsequent operation. The results also include the number of items exceeding the limit. A scan can result in no table data meeting the filter criteria.

Items should be retrieved from pages and decoded.

Use last zookeeper

Signed-off-by: Victor Castell <[email protected]>
@wolfeidau
Copy link
Contributor

Nice work, I will need this soon!

@abronan abronan merged commit ccf7df6 into master Apr 19, 2019
@abronan abronan deleted the dynamodb_list_all branch April 19, 2019 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants