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

Search pagination UI violates ES max limit #1223

Closed
amplifi opened this issue Mar 7, 2017 · 1 comment · Fixed by #1246
Closed

Search pagination UI violates ES max limit #1223

amplifi opened this issue Mar 7, 2017 · 1 comment · Fixed by #1246

Comments

@amplifi
Copy link
Contributor

amplifi commented Mar 7, 2017

Steps to reproduce the error

Search a project for a term with more than 10,000 search results. Example: search the "Uttaran Import" project for the term "e*"

Actual behavior

Platform returns 10,444 search results, violating the ES max limit of 10,000:

screen shot 2017-03-07 at 9 40 09 pm

If you set the results per page drop-down to show 100 results per page, then click the pagination arrows, all is fine through result # 10,000:

screen shot 2017-03-07 at 9 43 53 pm

But once you reach result # 10,001, the search UI dumps all results and fails with the error "Search is currently not available." The search query must be re-submitted.

screen shot 2017-03-07 at 9 44 01 pm

(Setting the drop-down to display 100 results per page is irrelevant; it just makes clicking through to result # 10,001 a lot less painful.)

Expected behavior

The platform UI must adhere to the ElasticSearch max results limit, currently set to the default of 10,000 as noted in Issue #1097 . Results should be capped at 10,000 with an appropriate message informing the user of this limit and (once implemented) the alternative option for project managers to export project data.

@dpalomino
Copy link

As agreed in the search check-in, @seav was going to work on this to fix it this week (thanks Eugene!).

seav added a commit that referenced this issue Mar 8, 2017
amplifi pushed a commit that referenced this issue Mar 10, 2017
laura-barluzzi pushed a commit to laura-barluzzi/cadasta-platform that referenced this issue Mar 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants