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
Function score multi values #5940
Conversation
@@ -36,6 +36,7 @@ | |||
import org.elasticsearch.index.fielddata.GeoPointValues; | |||
import org.elasticsearch.index.fielddata.IndexGeoPointFieldData; | |||
import org.elasticsearch.index.fielddata.IndexNumericFieldData; | |||
import org.elasticsearch.index.fielddata.fieldcomparator.SortMode; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should now move this mode into a util class maybe in org.elasticsearch.search.MultiValueMode
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
Decay functions currently only use the first value in a field that contains multiple values to compute the distance to the origin. Instead, it should consider all distances if more values are in the field and then use one of min/max/sum/avg which is defined by the user.
Thanks for the quick review! I updated the branch, added a commit for each comment. |
LGTM. |
LGTM |
Decay functions currently only use the first value in a field that contains multiple values to compute the distance to the origin. Instead, it should consider all distances if more values are in the field and then use one of min/max/sum/avg which is defined by the user. Relates to #3960 closes #5940
Decay functions currently only use the first value in a field that contains
multiple values to compute the distance to the origin. Instead, they should
consider all distances if more values are in the field and then use
one of min/max/sum/avg which is defined by the user.
closes #3960