Skip to content

Commit 2eaf753

Browse files
committed
[Tests] Removing skipping tests in search rest tests
After backporting the script_field soft limit to the 6.x branches, this test can now also run in a mixed cluster. Relates to #26598 enter the commit message for your changes. Lines starting
1 parent 7be5ee5 commit 2eaf753

File tree

2 files changed

+3
-12
lines changed

2 files changed

+3
-12
lines changed

core/src/main/java/org/elasticsearch/index/IndexSettings.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ public final class IndexSettings {
101101

102102
/**
103103
* Index setting describing the maximum value of allowed `script_fields`that can be retrieved
104-
* per search request. The default maximum of 50 is defensive for the reason that retrieving
104+
* per search request. The default maximum of 32 is defensive for the reason that retrieving
105105
* script fields is a costly operation.
106106
*/
107107
public static final Setting<Integer> MAX_SCRIPT_FIELDS_SETTING =

rest-api-spec/src/main/resources/rest-api-spec/test/search/30_limits.yml

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ setup:
55
body:
66
settings:
77
index.max_docvalue_fields_search: 2
8+
index.max_script_fields: 2
89

910
- do:
1011
index:
@@ -72,21 +73,11 @@ setup:
7273

7374
---
7475
"Script_fields size limit":
75-
- skip:
76-
version: " - 6.99.99"
77-
reason: soft limit for script_fields only available as of 7.0.0
78-
79-
- do:
80-
indices.create:
81-
index: test_2
82-
body:
83-
settings:
84-
index.max_script_fields: 2
8576

8677
- do:
8778
catch: /Trying to retrieve too many script_fields\. Must be less than or equal to[:] \[2\] but was \[3\]\. This limit can be set by changing the \[index.max_script_fields\] index level setting\./
8879
search:
89-
index: test_2
80+
index: test_1
9081
body:
9182
query:
9283
match_all: {}

0 commit comments

Comments
 (0)