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.

Sign up for GitHub

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

Jump to bottom

Reload analyzers should include analyzer names in response #43804

Closed
cbuescher opened this issue Jul 1, 2019 · 1 comment · Fixed by #43813
Closed

Reload analyzers should include analyzer names in response #43804

cbuescher opened this issue Jul 1, 2019 · 1 comment · Fixed by #43813
Assignees
@cbuescher
Labels
>enhancement :Search Relevance/Analysis How text is split into tokens Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v7.3.0 v8.0.0-alpha1

Comments

@cbuescher
Copy link
Member

Currently the repsonse of the "_reload_search_analyzer" endpoint contains the index names and nodeIds of indices were analyzers reloading was triggered. We should additionally include the name of the search-time analyzers that were potentially reloaded. I just spent some time wondering why some change to the synonyms file wasn't picked up by a reload where the cause was that the analyzers I used wasn't declared as search-time "updateable". Having the analyzer name in the response would have sped up that debugging process I think.
@cbuescher cbuescher self-assigned this Jul 1, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search

cbuescher pushed a commit to cbuescher/elasticsearch that referenced this issue Jul 1, 2019
Return reloaded analyzers in _reload_search_ananlyzer response
0024968 
Currently the repsonse of the "_reload_search_analyzer" endpoint contains the
index names and nodeIds of indices were analyzers reloading was triggered. This
change add the names of the search-time analyzers that were reloaded.

Closes elastic#43804
@cbuescher cbuescher mentioned this issue Jul 1, 2019
Merged
cbuescher pushed a commit to cbuescher/elasticsearch that referenced this issue Jul 2, 2019
Return reloaded analyzers in _reload_search_ananlyzer response
3578870 
Currently the repsonse of the "_reload_search_analyzer" endpoint contains the
index names and nodeIds of indices were analyzers reloading was triggered. This
change add the names of the search-time analyzers that were reloaded.

Closes elastic#43804
rjernst pushed a commit to rjernst/elasticsearch that referenced this issue Jul 2, 2019
Return reloaded analyzers in _reload_search_ananlyzer response (elast…
3cc222e 
…ic#43813)

Currently the repsonse of the "_reload_search_analyzer" endpoint contains the
index names and nodeIds of indices were analyzers reloading was triggered. This
change add the names of the search-time analyzers that were reloaded.

Closes elastic#43804
cbuescher pushed a commit that referenced this issue Jul 2, 2019
Return reloaded analyzers in _reload_search_ananlyzer response (#43813)
31cf96e 
Currently the repsonse of the "_reload_search_analyzer" endpoint contains the
index names and nodeIds of indices were analyzers reloading was triggered. This
change add the names of the search-time analyzers that were reloaded.

Closes #43804
@Mpdreamz Mpdreamz mentioned this issue Aug 7, 2019
Closed
16 tasks
@javanna javanna added the Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch label Jul 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cbuescher cbuescher

Labels
>enhancement :Search Relevance/Analysis How text is split into tokens Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v7.3.0 v8.0.0-alpha1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Return reloaded analyzers in _reload_search_ananlyzer response cbuescher/elasticsearch
4 participants
@javanna @jakelandis @cbuescher @elasticmachine