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

pagedCount() #259

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

pagedCount() #259

wants to merge 4 commits into from

Conversation

zgrguric
Copy link

@zgrguric zgrguric commented Oct 5, 2022

count() only counts correctly when scan or query is within 1MB dynamo limit or limit is explicitly set.

pagedCount() returns lastEvaluatedKey with count result on this page, if its != null then next count query can be executed

This method returns detailed counts and evaluated object which can be used for paginated count.
@scrutinizer-notifier
Copy link

The inspection completed: 1 updated code elements

@nelson6e65 nelson6e65 self-assigned this Apr 18, 2023
@nelson6e65 nelson6e65 added the missing tests Feature or bug that needs failing tests to replicate it label Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
missing tests Feature or bug that needs failing tests to replicate it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants