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

Remove Index Reader warmer introduced in 0.90.6 #4078

Closed
kimchy opened this issue Nov 4, 2013 · 0 comments
Closed

Remove Index Reader warmer introduced in 0.90.6 #4078

kimchy opened this issue Nov 4, 2013 · 0 comments
Labels
>regression v0.90.7

Comments

@kimchy
Copy link
Member

kimchy commented Nov 4, 2013

Remove the automatic index warmer introduced in 0.90.6 release, since it create the following challenges:

  • it automatically load all the norms for all fields. This should be an opt in feature similar to the new loading feature in field data. Will open a separate issue for it.
  • It automatically loads all doc values for all fields (if they have it), overriding effectively the loading option of field data when its backed by doc values.
@kimchy kimchy closed this as completed in 5f170cb Nov 4, 2013
kimchy added a commit that referenced this issue Nov 4, 2013
@kimchy
Remove Index Reader warmer introduced in 0.90.6
90090f8 
It create the following challenges:
- it automatically load all the norms for all fields. This should be an opt in feature similar to the new loading feature in field data. Will open a separate issue for it.
- It automatically loads all doc values for all fields (if they have it), overriding effectively the loading option of field data when its backed by doc values.

closes #4078
@clintongormley clintongormley mentioned this issue Nov 5, 2013
Closed
mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015
@kimchy
Remove Index Reader warmer introduced in 0.90.6
bdbcff0 
It create the following challenges:
- it automatically load all the norms for all fields. This should be an opt in feature similar to the new loading feature in field data. Will open a separate issue for it.
- It automatically loads all doc values for all fields (if they have it), overriding effectively the loading option of field data when its backed by doc values.

closes elastic#4078
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
>regression v0.90.7
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@kimchy