New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change default similarity to BM25 #18948

Merged
merged 1 commit into from Jun 21, 2016

Conversation

Projects
None yet
2 participants
@jimczi
Member

jimczi commented Jun 17, 2016

The default similarity was set to classic which refers to TFIDF and has not been moved after the upgrade to Lucene 6.

Though moving to BM25 could have some downside for queries that relies on coordination factor (match_query, multi_match_query) ?
relates #18944

@jpountz

View changes

Show outdated Hide outdated docs/reference/index-modules/similarity.asciidoc
@jimczi

This comment has been minimized.

Show comment
Hide comment
@jimczi

jimczi Jun 21, 2016

Member

@jpountz I rephrased the documentation. Can you take a look please ?

Member

jimczi commented Jun 21, 2016

@jpountz I rephrased the documentation. Can you take a look please ?

@jpountz

This comment has been minimized.

Show comment
Hide comment
@jpountz

jpountz Jun 21, 2016

Contributor

LGTM

Contributor

jpountz commented Jun 21, 2016

LGTM

Change default similarity to BM25
The default similarity was set to `classic` which refers to TFIDF and has not been moved after the upgrade to Lucene 6.

Though moving to BM25 could have some downside for queries that relies on coordination factor (match_query, multi_match_query) ?

relates #18944

@jimczi jimczi merged commit cc91014 into elastic:master Jun 21, 2016

1 check passed

CLA Commit author is a member of Elasticsearch
Details

@jimczi jimczi deleted the jimczi:bm25 branch Jun 21, 2016

@jimczi

This comment has been minimized.

Show comment
Hide comment
@jimczi

jimczi Jun 21, 2016

Member

Thanks @jpountz !

Member

jimczi commented Jun 21, 2016

Thanks @jpountz !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment