Merge pull request pelias#1376 from pelias/constant_score_filter

constant_score subqueries should use "filter" not "query"
davetha · Nov 4, 2019 · 4f8c9eb · 4f8c9eb
2 parents 8b881da + c34a0b3
commit 4f8c9eb
Show file tree

Hide file tree

Showing 17 changed files with 17 additions and 17 deletions.
diff --git a/query/view/ngrams_last_token_only.js b/query/view/ngrams_last_token_only.js
@@ -29,7 +29,7 @@ module.exports = function( vs ){
   // return the view rendered using the copy
   return {
     'constant_score': {
-      'query': ngrams_strict( vsCopy )
+      'filter': ngrams_strict( vsCopy )
     }
   };
 };
diff --git a/query/view/ngrams_last_token_only_multi.js b/query/view/ngrams_last_token_only_multi.js
@@ -37,7 +37,7 @@ module.exports = function (adminFields){
     // return the view rendered using the copy
     return {
       'constant_score': {
-        'query': rendered
+        'filter': rendered
       }
     };
   };

diff --git a/test/unit/fixture/autocomplete_boundary_country.js b/test/unit/fixture/autocomplete_boundary_country.js
@@ -3,7 +3,7 @@ module.exports = {
     'bool': {
       'must': [{
         'constant_score': {
-          'query': {
+          'filter': {
             'match_phrase': {
               'name.default': {
                 'analyzer': 'peliasQuery',

diff --git a/test/unit/fixture/autocomplete_boundary_gid.js b/test/unit/fixture/autocomplete_boundary_gid.js
@@ -3,7 +3,7 @@ module.exports = {
     'bool': {
       'must': [{
         'constant_score': {
-          'query': {
+          'filter': {
             'match_phrase': {
               'name.default': {
                 'analyzer': 'peliasQuery',

diff --git a/test/unit/fixture/autocomplete_linguistic_bbox_san_francisco.js b/test/unit/fixture/autocomplete_linguistic_bbox_san_francisco.js
@@ -3,7 +3,7 @@ module.exports = {
     'bool': {
       'must': [{
         'constant_score': {
-          'query': {
+          'filter': {
             'match_phrase': {
               'name.default': {
                 'analyzer': 'peliasQuery',

diff --git a/test/unit/fixture/autocomplete_linguistic_circle_san_francisco.js b/test/unit/fixture/autocomplete_linguistic_circle_san_francisco.js
@@ -4,7 +4,7 @@ module.exports = {
       'must': [
         {
           'constant_score': {
-            'query': {
+            'filter': {
               'match_phrase': {
                 'name.default': {
                   'analyzer': 'peliasQuery',

diff --git a/test/unit/fixture/autocomplete_linguistic_focus.js b/test/unit/fixture/autocomplete_linguistic_focus.js
@@ -3,7 +3,7 @@ module.exports = {
     'bool': {
       'must': [{
         'constant_score': {
-          'query': {
+          'filter': {
             'match_phrase': {
               'name.default': {
                 'analyzer': 'peliasQuery',

diff --git a/test/unit/fixture/autocomplete_linguistic_focus_null_island.js b/test/unit/fixture/autocomplete_linguistic_focus_null_island.js
@@ -3,7 +3,7 @@ module.exports = {
     'bool': {
       'must': [{
         'constant_score': {
-          'query': {
+          'filter': {
             'match_phrase': {
               'name.default': {
                 'analyzer': 'peliasQuery',

diff --git a/test/unit/fixture/autocomplete_linguistic_multiple_tokens.js b/test/unit/fixture/autocomplete_linguistic_multiple_tokens.js
@@ -13,7 +13,7 @@ module.exports = {
       },
       {
         'constant_score': {
-          'query': {
+          'filter': {
             'multi_match': {
               'fields': [
                 'parent.country.ngram^1',

diff --git a/test/unit/fixture/autocomplete_linguistic_multiple_tokens_complete_numeric.js b/test/unit/fixture/autocomplete_linguistic_multiple_tokens_complete_numeric.js
@@ -13,7 +13,7 @@ module.exports = {
       },
       {
         'constant_score': {
-          'query': {
+          'filter': {
             'match_phrase': {
               'name.default': {
                 'analyzer': 'peliasQuery',

diff --git a/test/unit/fixture/autocomplete_linguistic_one_char_token.js b/test/unit/fixture/autocomplete_linguistic_one_char_token.js
@@ -3,7 +3,7 @@ module.exports = {
     'bool': {
       'must': [{
         'constant_score': {
-          'query': {
+          'filter': {
             'match_phrase': {
               'name.default': {
                 'analyzer': 'peliasQuery',

diff --git a/test/unit/fixture/autocomplete_linguistic_only.js b/test/unit/fixture/autocomplete_linguistic_only.js
@@ -3,7 +3,7 @@ module.exports = {
     'bool': {
       'must': [{
         'constant_score': {
-          'query': {
+          'filter': {
             'match_phrase': {
               'name.default': {
                 'analyzer': 'peliasQuery',

diff --git a/test/unit/fixture/autocomplete_linguistic_three_char_token.js b/test/unit/fixture/autocomplete_linguistic_three_char_token.js
@@ -3,7 +3,7 @@ module.exports = {
     'bool': {
       'must': [{
         'constant_score': {
-          'query': {
+          'filter': {
             'match_phrase': {
               'name.default': {
                 'analyzer': 'peliasQuery',

diff --git a/test/unit/fixture/autocomplete_linguistic_two_char_token.js b/test/unit/fixture/autocomplete_linguistic_two_char_token.js
@@ -3,7 +3,7 @@ module.exports = {
     'bool': {
       'must': [{
         'constant_score': {
-          'query': {
+          'filter': {
             'match_phrase': {
               'name.default': {
                 'analyzer': 'peliasQuery',

diff --git a/test/unit/fixture/autocomplete_with_category_filtering.js b/test/unit/fixture/autocomplete_with_category_filtering.js
@@ -3,7 +3,7 @@ module.exports = {
     'bool': {
       'must': [{
         'constant_score': {
-          'query': {
+          'filter': {
             'match_phrase': {
               'name.default': {
                 'analyzer': 'peliasQuery',

diff --git a/test/unit/fixture/autocomplete_with_layer_filtering.js b/test/unit/fixture/autocomplete_with_layer_filtering.js
@@ -3,7 +3,7 @@ module.exports = {
     'bool': {
       'must': [{
         'constant_score': {
-          'query': {
+          'filter': {
             'match_phrase': {
               'name.default': {
                 'analyzer': 'peliasQuery',

diff --git a/test/unit/fixture/autocomplete_with_source_filtering.js b/test/unit/fixture/autocomplete_with_source_filtering.js
@@ -3,7 +3,7 @@ module.exports = {
     'bool': {
       'must': [{
         'constant_score': {
-          'query': {
+          'filter': {
             'match_phrase': {
               'name.default': {
                 'analyzer': 'peliasQuery',