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

Allow AnalyzeRequestBuilder to be given an arbitrary text array #3023

Closed
brian-from-fl opened this issue May 10, 2013 · 2 comments
Closed

Allow AnalyzeRequestBuilder to be given an arbitrary text array #3023

brian-from-fl opened this issue May 10, 2013 · 2 comments
Assignees
@johtani
Labels
>enhancement :Search/Analysis How text is split into tokens v2.0.0-beta1

Comments

@brian-from-fl
Copy link

The AnalyzeRequestBuilder is useful for seeing exactly how a text string value will be analyzed. But when a document can contain multiple values, it's often difficult to guess the tokens and token positions that result when ES is indexing a multi-valued field.

It would be very helpful to have the AnalyzeRequestBuilder accept an array of String objects, or perhaps even a JSON document with some restrictions, so that a real-world multi-valued document field is more accurately modeled and analyzed.
@s1monw s1monw closed this as completed in 09fb226 May 14, 2013
s1monw added a commit that referenced this issue May 14, 2013
@s1monw
Raise search threadpool default size.
076a1ed 
The default size used to be 2x availableProcessors which seemed to
be a to lowish value in practice. 3x appeared to be a sweetspot for
most application. The default is now 3 x availableProcessors

Closes #3023
@s1monw
Copy link
Contributor

s1monw commented May 14, 2013

I referenced the wrong issue from this commit... reopening

@s1monw s1monw reopened this May 14, 2013
@clintongormley
Copy link

This could be supported when #5866 is implemented

@clintongormley clintongormley added the :Search/Analysis How text is split into tokens label Nov 29, 2014
@clintongormley clintongormley mentioned this issue Dec 24, 2014
Closed
@johtani johtani self-assigned this Dec 29, 2014
johtani added a commit to johtani/elasticsearch that referenced this issue Apr 28, 2015
@johtani
Analysis: Add multi-valued text support
1d1f894 
Add support array text as a multi-valued for AnalyzeRequestBuilder
Add support array text as a multi-valued for Analyze REST API
Add docs

Closes elastic#3023
@johtani johtani mentioned this issue Apr 28, 2015
Closed
johtani added a commit to johtani/elasticsearch that referenced this issue Apr 28, 2015
@johtani
Analysis: Add multi-valued text support
55c0f5b 
Add support array text as a multi-valued for AnalyzeRequestBuilder
Add support array text as a multi-valued for Analyze REST API
Add docs

Closes elastic#3023
johtani added a commit to johtani/elasticsearch that referenced this issue Apr 30, 2015
@johtani
Analysis: Add multi-valued text support
b471d19 
Fix some comments

Closes elastic#3023
johtani added a commit to johtani/elasticsearch that referenced this issue May 2, 2015
@johtani
Analysis: Add multi-valued text support
91c7abd 
Change names of IndicesAdminClient methods

Closes elastic#3023
johtani added a commit to johtani/elasticsearch that referenced this issue May 13, 2015
@johtani
Analysis: Add multi-valued text support
598f94c 
Remove bad naming method
Add prepareAnalyze() with no params
Add setText to AnalyzeRequestBuilder

Closes elastic#3023
johtani added a commit to johtani/elasticsearch that referenced this issue May 15, 2015
@johtani
Analysis: Add multi-valued text support
139bd34 
remove setSource JSON

Closes elastic#3023
@javanna javanna mentioned this issue May 27, 2015
Closed
mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015
@s1monw
Raise search threadpool default size.
b822bfe 
The default size used to be 2x availableProcessors which seemed to
be a to lowish value in practice. 3x appeared to be a sweetspot for
most application. The default is now 3 x availableProcessors

Closes elastic#3023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@johtani johtani

Labels
>enhancement :Search/Analysis How text is split into tokens v2.0.0-beta1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add multi-valued text support to the analyzer API johtani/elasticsearch
5 participants
@clintongormley @javanna @s1monw @johtani @brian-from-fl