Skip to content

Conversation

@nik9000
Copy link
Member

@nik9000 nik9000 commented Oct 20, 2020

This drops the boost parameter of the distance_feature query builder
internally, relying on our query building infrastructure to wrap the
query in a boosting query.

Relates to #63767

@nik9000 nik9000 added >bug :Search/Search Search-related issues that do not fall into other categories v8.0.0 v7.11.0 labels Oct 20, 2020
@elasticmachine
Copy link
Collaborator

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

@elasticmachine elasticmachine added the Team:Search Meta label for search team label Oct 20, 2020
This drops the `boost` parameter of the `distance_feature` query builder
internally, relying on our query building infrastructure to wrap the
query in a `boosting` query.

Relates to elastic#63767
Copy link
Contributor

@mayya-sharipova mayya-sharipova left a comment

Choose a reason for hiding this comment

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

@nik9000 Thanks for this, LGTM!

@nik9000 nik9000 merged commit f2bcc77 into elastic:master Oct 21, 2020
@nik9000
Copy link
Member Author

nik9000 commented Oct 21, 2020

Thanks for reviewing @mayya-sharipova !

nik9000 added a commit to nik9000/elasticsearch that referenced this pull request Oct 21, 2020
This drops the `boost` parameter of the `distance_feature` query builder
internally, relying on our query building infrastructure to wrap the
query in a `boosting` query.

Relates to elastic#63767
nik9000 added a commit that referenced this pull request Oct 21, 2020
This drops the `boost` parameter of the `distance_feature` query builder
internally, relying on our query building infrastructure to wrap the
query in a `boosting` query.

Relates to #63767
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug :Search/Search Search-related issues that do not fall into other categories Team:Search Meta label for search team v7.11.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants