Hard limit on from/size #9311

Closed
clintongormley opened this Issue Jan 15, 2015 · 2 comments

Projects

None yet

4 participants

@clintongormley
Member

Requesting a million hits, or page 100,000 is always a bad idea, but users may not be aware of this. We should impose a per-index limit on the maximum size + from that can be requested, eg 10,000.

This should not interfere with deep-scrolling.

@rjernst
Member
rjernst commented Jan 16, 2015

+1

@s1monw s1monw added v1.6.0 and removed v1.6.0 v1.5.0 labels Mar 17, 2015
@clintongormley clintongormley added v2.0.0 and removed v1.6.0 labels May 29, 2015
@nik9000 nik9000 self-assigned this Aug 28, 2015
@nik9000
Contributor
nik9000 commented Aug 28, 2015

We should make sure that the error message returns a reference to scan/scroll too.

I'm grabbing this because I keep hearing about it being trouble.

@nik9000 nik9000 removed the adoptme label Aug 28, 2015
@nik9000 nik9000 added a commit to nik9000/elasticsearch that referenced this issue Sep 10, 2015
@nik9000 nik9000 [search] Limit the size of the result window
Requesting a million hits, or page 100,000 is always a bad idea, but users
may not be aware of this. This adds a per-index limit on the maximum size +
from that can be requested which defaults to 10,000.

This should not interfere with deep-scrolling.

Closes #9311
e498196
@nik9000 nik9000 closed this in #13188 Sep 10, 2015
@nik9000 nik9000 added a commit that referenced this issue Sep 10, 2015
@nik9000 nik9000 [search] Limit the size of the result window
Requesting a million hits, or page 100,000 is always a bad idea, but users
may not be aware of this. This adds a per-index limit on the maximum size +
from that can be requested which defaults to 10,000.

This should not interfere with deep-scrolling.

Closes #9311
7b0a876
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment