Skip to content

Commit

Permalink
Default include_type_name to false in the yml test harness. (#38058)
Browse files Browse the repository at this point in the history
This PR removes the temporary change we made to the yml test harness in #37285
to automatically set `include_type_name` to `true` in index creation requests
if it's not already specified. This is possible now that the vast majority of
index creation requests were updated to be typeless in #37611. A few additional
tests also needed updating here.

Additionally, this PR updates the test harness to set `include_type_name` to
`false` in index creation requests when communicating with 6.x nodes. This
mirrors the logic added in #37611 to allow for typeless document write requests
in test set-up code. With this update in place, we can remove many references
to `include_type_name: false` from the yml tests.
  • Loading branch information
jtibshirani authored Feb 1, 2019
1 parent 9350da9 commit c2e9d13
Show file tree
Hide file tree
Showing 203 changed files with 515 additions and 740 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
"Empty Bucket Aggregation":
- do:
indices.create:
include_type_name: false
index: empty_bucket_idx
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ setup:

- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ setup:

- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@

- do:
indices.create:
include_type_name: false
index: test_deprecated_htmlstrip
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
"validate query with synonyms":
- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
# versions in the same position.
- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
"ngram search":
- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down Expand Up @@ -44,7 +43,6 @@
"testNGramCopyField":
- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
"ngram highlighting":
- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
"Test query string with snowball":
- do:
indices.create:
include_type_name: false
index: test
body:
mappings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
"Test common terms query with stacked tokens":
- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down Expand Up @@ -220,7 +219,6 @@
"Test match query with synonyms - see #3881 for extensive description of the issue":
- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
setup:
- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
setup:
- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
"suggestions with synonyms":
- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
# because there are no token filters that support payloads in core.
- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -127,14 +127,12 @@
body: >
{
"mappings" : {
"test" : {
"properties" : {
"geoip.location" : {
"type": "geo_point"
}
}
}
}
}
- match: { acknowledged: true }

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
setup:
- do:
indices.create:
include_type_name: false
index: test123
body:
mappings:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
setup:
- do:
indices.put_template:
include_type_name: false
name: index_template
body:
index_patterns: test-*
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
setup:
- do:
indices.create:
include_type_name: false
index: test
body:
mappings:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
setup:
- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
features: ["headers"]
- do:
indices.create:
include_type_name: false
index: test
body:
mappings:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
setup:
- do:
indices.create:
include_type_name: false
index: my-index
body:
mappings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ setup:
reason: "moving_fn added in 6.4.0"
- do:
indices.create:
include_type_name: false
index: test
body:
mappings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,6 @@ setup:
"Random functions":
- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down Expand Up @@ -136,7 +135,6 @@ setup:
"Decay geo functions":
- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ setup:

- do:
indices.create:
include_type_name: false
index: test
body:
mappings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ setup:

- do:
indices.create:
include_type_name: false
index: test-index
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ setup:

- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ setup:

- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
setup:
- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ setup:

- do:
indices.create:
include_type_name: false
index: test-index
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
setup:
- do:
indices.create:
include_type_name: false
index: test
body:
mappings:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
setup:
- do:
indices.create:
include_type_name: false
index: test
body:
mappings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
"Test percolator basics via rest":
- do:
indices.create:
include_type_name: false
index: queries_index
body:
mappings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -166,9 +166,8 @@
index: test
body:
mappings:
test:
_source:
enabled: false
_source:
enabled: false
- do:
index:
index: test
Expand All @@ -178,7 +177,7 @@
indices.refresh: {}

- do:
catch: /\[test\]\[test\]\[1\] didn't store _source/
catch: /\[test\]\[_doc\]\[1\] didn't store _source/
reindex:
body:
source:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,18 +4,16 @@ setup:
index: source
body:
mappings:
doc:
properties:
join_field: { "type": "join", "relations": { "parent": "child", "child": "grand_child" } }
properties:
join_field: { "type": "join", "relations": { "parent": "child", "child": "grand_child" } }

- do:
indices.create:
index: dest
body:
mappings:
doc:
properties:
join_field: { "type": "join", "relations": { "parent": "child", "child": "grand_child" } }
properties:
join_field: { "type": "join", "relations": { "parent": "child", "child": "grand_child" } }

- do:
index:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,9 @@
index: test
body:
mappings:
place:
properties:
name:
type: keyword
properties:
name:
type: keyword
- do:
index:
index: test
Expand All @@ -18,7 +17,6 @@

- do:
indices.put_mapping:
include_type_name: false
index: test
body:
properties:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
"Index ICU content":
- do:
indices.create:
include_type_name: false
index: test
body:
settings:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,10 @@
index: test
body:
mappings:
type:
properties:
text:
type: text
analyzer: kuromoji
properties:
text:
type: text
analyzer: kuromoji

- do:
index:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,10 @@
index: test
body:
mappings:
type:
properties:
text:
type: text
analyzer: nori
properties:
text:
type: text
analyzer: nori

- do:
index:
Expand Down
Loading

0 comments on commit c2e9d13

Please sign in to comment.