Skip to content

Fix highlighting when no field is specified#4519

Merged
w33ble merged 3 commits intoelastic:masterfrom
lukasolson:field-highlight
Aug 4, 2015
Merged

Fix highlighting when no field is specified#4519
w33ble merged 3 commits intoelastic:masterfrom
lukasolson:field-highlight

Conversation

@lukasolson
Copy link
Contributor

Closes #4494.

With the upgrade to ES 2.0, there were some changes to highlighting, namely the default setting for require_field_match.

From the 1.6 documentation:

require_field_match can be set to true which will cause a field to be highlighted only if a query matched that field. false means that terms are highlighted on all requested fields regardless if the query matches specifically on them.

From the 2.0 documentation:

require_field_match can be set to false which will cause any field to be highlighted regardless of whether the query matched specifically on them. The default behaviour is true, meaning that only fields that hold a query match will be highlighted.

@spalger
Copy link
Contributor

spalger commented Jul 30, 2015

Submitted lukasolson#6 which merged master

@w33ble
Copy link
Contributor

w33ble commented Aug 4, 2015

Nice find. LGTM!

w33ble added a commit that referenced this pull request Aug 4, 2015
Fix highlighting when no field is specified
@w33ble w33ble merged commit 44a9f70 into elastic:master Aug 4, 2015
@lukasolson lukasolson deleted the field-highlight branch February 11, 2016 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants