Skip to content

_analyze API fails if char_filters is set in body. #15257

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

Closed
masaruh opened this issue Dec 5, 2015 · 3 comments
Closed

_analyze API fails if char_filters is set in body. #15257

masaruh opened this issue Dec 5, 2015 · 3 comments
Labels
>bug :Search Relevance/Analysis How text is split into tokens Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch

Comments

@masaruh
Copy link
Contributor

masaruh commented Dec 5, 2015

If it's passed as URL parameter, it works.

It looks _analyze API sets char_filters as token_filters:
https://github.com/elastic/elasticsearch/blob/master/core/src/main/java/org/elasticsearch/rest/action/admin/indices/analyze/RestAnalyzeAction.java#L132

@masaruh masaruh added :Search Relevance/Analysis How text is split into tokens :Core/Infra/REST API REST infrastructure and utilities labels Dec 5, 2015
@clintongormley clintongormley added >bug and removed :Core/Infra/REST API REST infrastructure and utilities labels Dec 5, 2015
@clintongormley
Copy link
Contributor

@johtani want to take a look?

@johtani
Copy link
Contributor

johtani commented Dec 10, 2015

@clintongormley Yes. And I will fix in #11660.

johtani added a commit that referenced this issue Dec 10, 2015
add explain option
fix char_filter bug

Closes #11076 #15257
johtani added a commit that referenced this issue Dec 10, 2015
add explain option
fix char_filter bug

Closes #11076 #15257
@johtani
Copy link
Contributor

johtani commented Dec 10, 2015

Closed by master : fab4439
2.x : f427cf4

@johtani johtani closed this as completed Dec 10, 2015
@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
Labels
>bug :Search Relevance/Analysis How text is split into tokens Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch
Projects
None yet
Development

No branches or pull requests

4 participants