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

pagination - total_num_found == num_returned #477

Open
elrayle opened this issue Feb 16, 2022 · 1 comment
Open

pagination - total_num_found == num_returned #477

elrayle opened this issue Feb 16, 2022 · 1 comment
Assignees

Comments

@elrayle
Copy link
Member

elrayle commented Feb 16, 2022

Description

The total_records as reported by the cache should be the total number of results that lucene believes matches the query.

Expected

maxRecords=5 & lucene reports 81 total matches ==> total_records should be 81

Actual

maxRecords=5 & lucene reports 81 total matches ==> total_records is 5

Proposed Solution

The cache will use a 6th query to get an approximation of the total count. The cache will return a number between the number returned and 1000. Pagination will show up in Sinopia if total count > requested.

@elrayle
Copy link
Member Author

elrayle commented Apr 27, 2022

@eichmann Plans to work on it this week. Not sure if there will be a performance impact of the approach on large result sets.

Going to try to run a test query that doesn't actually return results.

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

No branches or pull requests

2 participants