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

All field uses wrong setting for `term vectors` #4581

Closed
s1monw opened this Issue Jan 2, 2014 · 0 comments

Comments

Projects
None yet
1 participant
@s1monw
Contributor

s1monw commented Jan 2, 2014

In the all field mapper the settings that is used is store_term_vector but it should be store_term_vectors.

@ghost ghost assigned s1monw Jan 2, 2014

@s1monw s1monw closed this in beaa915 Jan 2, 2014

s1monw added a commit that referenced this issue Jan 2, 2014

Simulate the entire toXContent instead of special caseing
Today we try to detect if we need to generate the mapping or not in
the all mapper. This is error prone since it misses conditions if not
explicitly added. We should rather similate the generation instead.

This commit also adds a random test to check if the settings
of the all field mapper are correctly applied.

Closes #4579
Closes #4581

brusic added a commit to brusic/elasticsearch that referenced this issue Jan 19, 2014

Simulate the entire toXContent instead of special caseing
Today we try to detect if we need to generate the mapping or not in
the all mapper. This is error prone since it misses conditions if not
explicitly added. We should rather similate the generation instead.

This commit also adds a random test to check if the settings
of the all field mapper are correctly applied.

Closes #4579
Closes #4581

mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015

Simulate the entire toXContent instead of special caseing
Today we try to detect if we need to generate the mapping or not in
the all mapper. This is error prone since it misses conditions if not
explicitly added. We should rather similate the generation instead.

This commit also adds a random test to check if the settings
of the all field mapper are correctly applied.

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