-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
Pagination for aggregation queries is currently broken, because we are getting the total number of rows based on the Content-Range that is returned by postgrest, which describes the total number of rows in the table, whereas we are looking for the number of rows in the requested aggregation.
Examples:
?birth__less=1980has the rightContent-Range?birth__groupby&birth__countgives a total of 1000 rows (the number of rows of the unfiltered table)?birth__groupby&birth__count&score__less=0.3gives the sametotalas?score__less=0.3
We are not getting the total number of rows for the aggregated query unless we don't specify the offset and count the number of results "manually", which would mean another (aggregation) query for each aggregation query. Do we actual need the total in these cases?
For now the considered patch is:
- to stop displaying the (wrong) total for aggregation queries
- to slightly refactor the way we compute the
next_page
Metadata
Metadata
Assignees
Labels
No labels