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 level boosting #51703

Open
1 of 3 tasks
mayya-sharipova opened this issue Jan 30, 2020 · 5 comments
Open
1 of 3 tasks

Remove index level boosting #51703

mayya-sharipova opened this issue Jan 30, 2020 · 5 comments
Labels
>deprecation :Search Foundations/Mapping Index mappings, including merging and defining field types Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch

Comments

@mayya-sharipova
Copy link
Contributor

mayya-sharipova commented Jan 30, 2020 •
edited
Loading

Currently there is no index level boosting,
as boosting is only applied during query time.
But we still allow to define boosting in mapping, which we should remove.

  • Remove mentions to index time boosting in the docs.
  • Remove boost option in the index mapping.
  • error when boost option is supplied on 8.x indexes
  • issue a deprecation warning when boost option is supplied on 7.x indexes.
  • Upgrade path. update documentation that old indexes should be reindexed without index boost parameters, and queries should be updated accordingly.
@mayya-sharipova mayya-sharipova added the :Search Foundations/Mapping Index mappings, including merging and defining field types label Jan 30, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search (:Search/Mapping)

@mayya-sharipova mayya-sharipova mentioned this issue Jan 30, 2020
Merged
@jimczi
Copy link
Contributor

jimczi commented Jan 30, 2020

Upgrade path. update documentation that old indexes should be reindexed without index boost parameters, and queries should be updated accordingly.

There's no need to reindex, we could just remove the parameter automatically in 8.x (as part of the upgrade).

Other than that +1 for the plan!

@rjernst rjernst added the Team:Search Meta label for search team label May 4, 2020
@arun-krishnamoorthy
Copy link

What is the reason for the warning in version 7.x , does that mean elastic v 7.x still supports index level boosting,i recently created a index in 7.x and the index mapping was accepting boost parameters ,Please clarify ?

Mpdreamz added a commit to elastic/elasticsearch-net that referenced this issue Oct 2, 2020
@Mpdreamz
Remove index level boosting
10c8a57 
Relates: elastic/elasticsearch#51703

Index level boosting existed in the mapping was not being used at query
time. This removes all boosting in anticipation of the full removal.
@Mpdreamz Mpdreamz mentioned this issue Oct 2, 2020
Merged
Mpdreamz added a commit to elastic/elasticsearch-net that referenced this issue Oct 2, 2020
@Mpdreamz
Remove index level boosting (#5045)
f305942 
Relates: elastic/elasticsearch#51703

Index level boosting existed in the mapping was not being used at query
time. This removes all boosting in anticipation of the full removal.
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-search (Team:Search)

@javanna javanna added Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch and removed Team:Search Meta label for search team labels Jul 16, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-search-foundations (Team:Search Foundations)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
>deprecation :Search Foundations/Mapping Index mappings, including merging and defining field types Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@rjernst @javanna @mayya-sharipova @elasticmachine @jimczi @arun-krishnamoorthy @elasticsearchmachine