Removes yaml circuit-breaker tests for geoshape geogrid aggs (#64420)
These tests were added to do a proper end-to-end test of the memory usage of the geotile_grid and geohash_grid aggregations on `geo_shape` fields. Although this was asperational, the truth is — the test environment does not run these aggregations in isolation. This means that the memory overhead is variable and too flaky to rely on over time. The unit tests for circuit-breaking remain. Closes #63158.
This commit is contained in:
parent
0c28bccbc8
commit
ff829bf197
|
@ -28,6 +28,5 @@ restResources {
|
||||||
|
|
||||||
testClusters.all {
|
testClusters.all {
|
||||||
setting 'xpack.license.self_generated.type', 'trial'
|
setting 'xpack.license.self_generated.type', 'trial'
|
||||||
setting 'indices.breaker.request.limit', '25kb'
|
|
||||||
testDistribution = 'DEFAULT'
|
testDistribution = 'DEFAULT'
|
||||||
}
|
}
|
||||||
|
|
|
@ -55,53 +55,3 @@
|
||||||
- match: { aggregations.grid.buckets.0.doc_count: 4 }
|
- match: { aggregations.grid.buckets.0.doc_count: 4 }
|
||||||
- match: { aggregations.grid.buckets.1.key: "5/16/11" }
|
- match: { aggregations.grid.buckets.1.key: "5/16/11" }
|
||||||
- match: { aggregations.grid.buckets.1.doc_count: 2 }
|
- match: { aggregations.grid.buckets.1.doc_count: 2 }
|
||||||
|
|
||||||
---
|
|
||||||
"Test geotile_grid aggregation circuit breaker":
|
|
||||||
- do:
|
|
||||||
indices.create:
|
|
||||||
index: locations
|
|
||||||
body:
|
|
||||||
settings:
|
|
||||||
number_of_shards: 1
|
|
||||||
number_of_replicas: 0
|
|
||||||
mappings:
|
|
||||||
properties:
|
|
||||||
location:
|
|
||||||
type: geo_shape
|
|
||||||
|
|
||||||
- do:
|
|
||||||
index:
|
|
||||||
index: locations
|
|
||||||
id: box
|
|
||||||
body: { location: "POLYGON((-173.671875 -77.8418477505252,173.32031249999997 -77.8418477505252,173.32031249999997 83.52016238353205,-173.671875 83.52016238353205,-173.671875 -77.8418477505252))" }
|
|
||||||
|
|
||||||
- do:
|
|
||||||
indices.refresh: {}
|
|
||||||
|
|
||||||
- do:
|
|
||||||
catch: /data for \[<agg \[grid\]>\] would be \[42760\/41.7kb\], which is larger than the limit of \[25600\/25kb\]/
|
|
||||||
search:
|
|
||||||
rest_total_hits_as_int: true
|
|
||||||
index: locations
|
|
||||||
size: 0
|
|
||||||
body:
|
|
||||||
aggs:
|
|
||||||
grid:
|
|
||||||
geotile_grid:
|
|
||||||
precision: 11
|
|
||||||
field: location
|
|
||||||
|
|
||||||
- do:
|
|
||||||
catch: /data for \[<agg \[grid\]>\] would be \[42760\/41.7kb\], which is larger than the limit of \[25600\/25kb\]/
|
|
||||||
search:
|
|
||||||
rest_total_hits_as_int: true
|
|
||||||
index: locations
|
|
||||||
size: 0
|
|
||||||
body:
|
|
||||||
aggs:
|
|
||||||
grid:
|
|
||||||
geotile_grid:
|
|
||||||
precision: 11
|
|
||||||
field: location
|
|
||||||
|
|
||||||
|
|
|
@ -57,52 +57,3 @@
|
||||||
- match: { aggregations.grid.buckets.1.doc_count: 2 }
|
- match: { aggregations.grid.buckets.1.doc_count: 2 }
|
||||||
- match: { aggregations.grid.buckets.2.key: "u155" }
|
- match: { aggregations.grid.buckets.2.key: "u155" }
|
||||||
- match: { aggregations.grid.buckets.2.doc_count: 1 }
|
- match: { aggregations.grid.buckets.2.doc_count: 1 }
|
||||||
|
|
||||||
---
|
|
||||||
"Test geohash_grid aggregation circuit breaker":
|
|
||||||
- do:
|
|
||||||
indices.create:
|
|
||||||
index: locations
|
|
||||||
body:
|
|
||||||
settings:
|
|
||||||
number_of_shards: 1
|
|
||||||
number_of_replicas: 0
|
|
||||||
mappings:
|
|
||||||
properties:
|
|
||||||
location:
|
|
||||||
type: geo_shape
|
|
||||||
|
|
||||||
- do:
|
|
||||||
index:
|
|
||||||
index: locations
|
|
||||||
id: box
|
|
||||||
body: { location: "POLYGON((-173.671875 -77.8418477505252,173.32031249999997 -77.8418477505252,173.32031249999997 83.52016238353205,-173.671875 83.52016238353205,-173.671875 -77.8418477505252))" }
|
|
||||||
|
|
||||||
- do:
|
|
||||||
indices.refresh: {}
|
|
||||||
|
|
||||||
- do:
|
|
||||||
catch: /data for \[<agg \[grid\]>\] would be \[30160\/29.4kb\], which is larger than the limit of \[25600\/25kb\]/
|
|
||||||
search:
|
|
||||||
rest_total_hits_as_int: true
|
|
||||||
index: locations
|
|
||||||
size: 0
|
|
||||||
body:
|
|
||||||
aggs:
|
|
||||||
grid:
|
|
||||||
geotile_grid:
|
|
||||||
precision: 10
|
|
||||||
field: location
|
|
||||||
|
|
||||||
- do:
|
|
||||||
catch: /data for \[<agg \[grid\]>\] would be \[30160\/29.4kb\], which is larger than the limit of \[25600\/25kb\]/
|
|
||||||
search:
|
|
||||||
rest_total_hits_as_int: true
|
|
||||||
index: locations
|
|
||||||
size: 0
|
|
||||||
body:
|
|
||||||
aggs:
|
|
||||||
grid:
|
|
||||||
geotile_grid:
|
|
||||||
precision: 10
|
|
||||||
field: location
|
|
||||||
|
|
Loading…
Reference in New Issue