Skip to content
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

Add support for Lucene SuggestStopFilter #3863

Merged
merged 1 commit into from
Oct 15, 2013

Conversation

spinscale
Copy link
Contributor

The suggest stop filter is an improved version of the stop filter, which
takes stopwords only into account if the last char of a query is a
whitespace. This allows you to keep stopwords, but to allow suggesting for
"a".

Example: Index document content "a word". You are now able to suggest for
"a" and get back results in the completion suggester, if the suggest stop
filter is used on the query side, but will not get back any results for
"a " as this is identified as a stopword.

@s1monw
Copy link
Contributor

s1monw commented Oct 9, 2013

Can we call this completion_stop rather than suggest_stop otherwise this might be confused to be useful for spellchecking? I also think we should include the documentation for this new filter in the commit?

@spinscale
Copy link
Contributor Author

Was thinking if it might be usable in match_phrase_prefix queries as well?

@s1monw
Copy link
Contributor

s1monw commented Oct 10, 2013

hmm maybe we do it totally different and make it an option on stop instead of adding a new token filter? like remove_trailing : true|false

@spinscale
Copy link
Contributor Author

Updated to work as an additional option for the existing stopfilter, instead of being an own one.
Also updated documentation.

@s1monw
Copy link
Contributor

s1monw commented Oct 15, 2013

LGTM

The suggest stop filter is an improved version of the stop filter, which
takes stopwords only into account if the last char of a query is a
whitespace. This allows you to keep stopwords, but to allow suggesting for
"a".

Example: Index document content "a word". You are now able to suggest for
"a" and get back results in the completion suggester, if the suggest stop
filter is used on the query side, but will not get back any results for
"a " as this is identified as a stopword.

The implementation allows to set the `remove_trailing` parameter for a
custom stop filter and thus use the suggest stop filter instead of the
standard stop filter.
@spinscale spinscale merged commit 4d19239 into elastic:master Oct 15, 2013
jakelandis added a commit to jakelandis/kibana that referenced this pull request Feb 8, 2019
This commit adopts Elasticsearch's typeless API for Logstash
central management.

Relates: elastic/elasticsearch#3863
jakelandis added a commit to jakelandis/logstash that referenced this pull request Feb 8, 2019
This commit adopts Elasticsearch's typeless API for central management.

Relates: elastic/elasticsearch#3863
alexwizp pushed a commit to elastic/kibana that referenced this pull request Mar 1, 2019
This commit adopts Elasticsearch's typeless API for Logstash
central management.

Relates: elastic/elasticsearch#3863
alexwizp pushed a commit to alexwizp/kibana that referenced this pull request Mar 1, 2019
This commit adopts Elasticsearch's typeless API for Logstash
central management.

Relates: elastic/elasticsearch#3863
alexwizp pushed a commit to alexwizp/kibana that referenced this pull request Mar 1, 2019
This commit adopts Elasticsearch's typeless API for Logstash
central management.

Relates: elastic/elasticsearch#3863
alexwizp added a commit to elastic/kibana that referenced this pull request Mar 1, 2019
This commit adopts Elasticsearch's typeless API for Logstash
central management.

Relates: elastic/elasticsearch#3863
alexwizp added a commit to elastic/kibana that referenced this pull request Mar 4, 2019
This commit adopts Elasticsearch's typeless API for Logstash
central management.

Relates: elastic/elasticsearch#3863
jakelandis added a commit to jakelandis/logstash that referenced this pull request Mar 6, 2019
This commit adopts Elasticsearch's typeless API for central management.

Relates: elastic/elasticsearch#3863
elasticsearch-bot pushed a commit to elastic/logstash that referenced this pull request Mar 7, 2019
This commit adopts Elasticsearch's typeless API for central management.

Relates: elastic/elasticsearch#3863

Fixes #10421
elasticsearch-bot pushed a commit to elastic/logstash that referenced this pull request Mar 7, 2019
This commit adopts Elasticsearch's typeless API for central management.

Relates: elastic/elasticsearch#3863

Fixes #10421
elasticsearch-bot pushed a commit to elastic/logstash that referenced this pull request Mar 7, 2019
This commit adopts Elasticsearch's typeless API for central management.

Relates: elastic/elasticsearch#3863

Fixes #10421
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants