Skip to content

[DOCS] Reformat stemmer token filter#55693

Merged
jrodewig merged 3 commits intoelastic:masterfrom
jrodewig:docs__reformat-stemmer-tokenfilter
Apr 24, 2020
Merged

[DOCS] Reformat stemmer token filter#55693
jrodewig merged 3 commits intoelastic:masterfrom
jrodewig:docs__reformat-stemmer-tokenfilter

Conversation

@jrodewig
Copy link
Copy Markdown
Contributor

Makes the following changes to the stemmer token filter docs:

  • Adds detailed analyze example
  • Rewrites parameter definitions
  • Adds custom analyzer example
  • Adds a language value for the estonian stemmer
  • Reorders the language values to show recommended algorithms first,
    followed by other values alphabetically

Makes the following changes to the `stemmer` token filter docs:

* Adds detailed analyze example
* Rewrites parameter definitions
* Adds custom analyzer example
* Adds a `language` value for the `estonian` stemmer
* Reorders the `language` values to show recommended algorithms first,
  followed by other values alphabetically
@jrodewig jrodewig added >docs General docs changes :Search Relevance/Analysis How text is split into tokens labels Apr 23, 2020
@jrodewig jrodewig requested a review from romseygeek April 23, 2020 20:01
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-search (:Search/Analysis)

@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-docs (>docs)

// This attribute prevents that substitution.
// See https://github.com/asciidoctor/asciidoctor/wiki/How-to-prevent-URLs-containing-formatting-characters-from-getting-mangled
:min-pt-stemmer-values-url: http://www.inf.ufrgs.br/~buriol/papers/Orengo_CLEF07.pdf
Provides algorithmic stemming for several languages, some with additional
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown
Contributor

@romseygeek romseygeek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @jrodewig

Arabic::

http://lucene.apache.org/core/4_9_0/analyzers-common/org/apache/lucene/analysis/ar/ArabicStemmer.html[*`arabic`*]
{lucene-analysis-docs}/ar/ArabicStemmer.html[*`arabic`*]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@jrodewig
Copy link
Copy Markdown
Contributor Author

@elasticmachine update branch

@jrodewig jrodewig merged commit bb9dbcb into elastic:master Apr 24, 2020
@jrodewig jrodewig deleted the docs__reformat-stemmer-tokenfilter branch April 24, 2020 15:08
jrodewig added a commit that referenced this pull request Apr 24, 2020
Makes the following changes to the `stemmer` token filter docs:

* Adds detailed analyze example
* Rewrites parameter definitions
* Adds custom analyzer example
* Adds a `language` value for the `estonian` stemmer
* Reorders the `language` values to show recommended algorithms first,
  followed by other values alphabetically
jrodewig added a commit that referenced this pull request Apr 24, 2020
Makes the following changes to the `stemmer` token filter docs:

* Adds detailed analyze example
* Rewrites parameter definitions
* Adds custom analyzer example
* Adds a `language` value for the `estonian` stemmer
* Reorders the `language` values to show recommended algorithms first,
  followed by other values alphabetically
jrodewig added a commit that referenced this pull request Apr 24, 2020
Makes the following changes to the `stemmer` token filter docs:

* Adds detailed analyze example
* Rewrites parameter definitions
* Adds custom analyzer example
* Adds a `language` value for the `estonian` stemmer
* Reorders the `language` values to show recommended algorithms first,
  followed by other values alphabetically
@jrodewig
Copy link
Copy Markdown
Contributor Author

Backport commits

master bb9dbcb
7.x e0a8adb
7.7 9ba83ef
7.6 7ffcb9e

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants