Skip to content

Conversation

@cbuescher
Copy link
Member

This PR adds documentation to the feature being worked on in #42669

@cbuescher cbuescher added >docs General docs changes :Search Relevance/Analysis How text is split into tokens labels Jun 4, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search

Copy link
Contributor

@jimczi jimczi left a comment

Choose a reason for hiding this comment

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

Thanks @cbuescher . I left some comments.

@@ -0,0 +1,56 @@
[[indices-reload-analyzers]]
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be marked as experimental.

Using the <<analyzer reload API,indices-reload-analyzers>>, you can trigger reloading of the
synonym definition. The contents of the configured synonyms file will be reloaded and the
synonyms definition the filter uses will be updated. Note that if you trying to use the above
analyzer as an index analyzer will result in an error.
Copy link
Contributor

Choose a reason for hiding this comment

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

I would extract the last sentence in a NOTE in order to make sure that users are aware of the limitation.

<2> Synonym filter is usable as a search_analyzer.

Calling the `_reload_search_analyzers` endpoint will now trigger reloading the
synonyms from the configured "synonym.txt" file.
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe add a NOTE regarding the fact that the reload is per node so you need to update your synonym.txt file on every node before calling reload.

@cbuescher cbuescher closed this Jun 5, 2019
@cbuescher
Copy link
Member Author

@jimczi I will also reopen this against the new feature branch I had to create and address your comments there.

@cbuescher cbuescher deleted the reloadable-analyzers-docs branch June 5, 2019 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>docs General docs changes :Search Relevance/Analysis How text is split into tokens

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants