Skip to content

Conversation

@pgomulka
Copy link
Contributor

@pgomulka pgomulka commented Jun 22, 2021

Previously removed #55078 indices_boost field is available when used
with rest api compatibility

relates #51816

Previouslly removed elastic#55078 indices_boost field is available when used
with rest api compatibility

relates elastic#51816
@pgomulka pgomulka added :Core/Infra/REST API REST infrastructure and utilities v8.0.0 labels Jun 22, 2021
@pgomulka pgomulka requested a review from jtibshirani June 22, 2021 12:38
@pgomulka pgomulka self-assigned this Jun 22, 2021
@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label Jun 22, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@pgomulka
Copy link
Contributor Author

pgomulka commented Jun 22, 2021

it was super easy to add, but if it is not needed we can discard it
It was marked as "not sure" and it was not removed in v7 yet.

Not sure we need REST compatibility, as it was deprecated back in 5.2

@pgomulka pgomulka added the :Search/Search Search-related issues that do not fall into other categories label Jun 22, 2021
@elasticmachine elasticmachine added the Team:Search Meta label for search team label Jun 22, 2021
@elasticmachine
Copy link
Collaborator

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

Copy link
Contributor

@jtibshirani jtibshirani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, it's self-contained and doesn't add too much complexity.

@pgomulka pgomulka merged commit 5d4dbea into elastic:master Jun 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/REST API REST infrastructure and utilities :Search/Search Search-related issues that do not fall into other categories Team:Core/Infra Meta label for core/infra team Team:Search Meta label for search team v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants