Skip to content

Move more token filters to analysis-common module#26042

Merged
martijnvg merged 1 commit intoelastic:masterfrom
martijnvg:move_more_token_filters_to_analyis-commons_module6
Aug 11, 2017
Merged

Move more token filters to analysis-common module#26042
martijnvg merged 1 commit intoelastic:masterfrom
martijnvg:move_more_token_filters_to_analyis-commons_module6

Conversation

@martijnvg
Copy link
Copy Markdown
Member

The following token filters were moved: arabic_stem, brazilian_stem, czech_stem, dutch_stem, french_stem, german_stem and russian_stem.

Relates to #23658

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This analyzer with czech stem token filter wasn't actually being used. Besides that the commented out test was redundant and the dependency on this token filter is unneeded in this test case.

The following token filters were moved: arabic_stem, brazilian_stem, czech_stem, dutch_stem, french_stem, german_stem and russian_stem.

Relates to elastic#23658
@martijnvg martijnvg force-pushed the move_more_token_filters_to_analyis-commons_module6 branch from 962f64d to 1146a35 Compare August 11, 2017 15:39
@martijnvg martijnvg merged commit 1146a35 into elastic:master Aug 11, 2017
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