Skip to content

Fix max score tracking with field collapsing#27122

Merged
jimczi merged 1 commit intoelastic:masterfrom
jimczi:field_collapsing_max_score
Oct 27, 2017
Merged

Fix max score tracking with field collapsing#27122
jimczi merged 1 commit intoelastic:masterfrom
jimczi:field_collapsing_max_score

Conversation

@jimczi
Copy link
Copy Markdown
Contributor

@jimczi jimczi commented Oct 26, 2017

This change makes sure that we track score when sort is set to relevancy only.
In this case we always track max score like normal search does.

Closes #23840

This change makes sure that we track score when sort is set to relevancy only.
In this case we always track max score like normal search does.

Closes elastic#23840
@jimczi jimczi added :Search/Search Search-related issues that do not fall into other categories >bug v6.1.0 v7.0.0 labels Oct 26, 2017
@jimczi jimczi merged commit 6625ecf into elastic:master Oct 27, 2017
@jimczi jimczi deleted the field_collapsing_max_score branch October 27, 2017 07:18
jimczi added a commit that referenced this pull request Oct 27, 2017
This change makes sure that we track score when sort is set to relevancy only.
In this case we always track max score like normal search does.

Closes #23840
jasontedor added a commit to olcbean/elasticsearch that referenced this pull request Oct 30, 2017
* master: (63 commits)
  [Docs] Fix note in bucket_selector
  [Docs] Fix indentation of examples (elastic#27168)
  [Docs] Clarify `span_not` query behavior for non-overlapping matches (elastic#27150)
  [Docs] Remove first person "I" from getting started (elastic#27155)
  [Docs] Correct link target for datatype murmur3 (elastic#27143)
  Fix division by zero in phrase suggester that causes assertion to fail
  Enable Docstats with totalSizeInBytes for 6.1.0
  Adds average document size to DocsStats (elastic#27117)
  Upgrade Painless from ANTLR 4.5.1-1 to  ANTLR 4.5.3. (elastic#27153)
  Exists template needs a template name (elastic#25988)
  [Tests] Fix occasional test failure due to two random values being the same
  Fix beidermorse phonetic token filter for unspecified `languageset` (elastic#27112)
  Fix max score tracking with field collapsing (elastic#27122)
  [Doc] Add Ingest CSV Processor Plugin to plugin as a community plugin (elastic#27105)
  Removed the beta tag from cross-cluster search
  fixed typo in ConstructingObjectParse (elastic#27129)
  Allow for the Painless Definition to have multiple instances (elastic#27096)
  Apply missing request options to the expand phase (elastic#27118)
  Only pull SegmentReader once in getSegmentInfo (elastic#27121)
  Fix BWC for discovery stats
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug :Search/Search Search-related issues that do not fall into other categories v6.1.0 v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants