Skip to content

Fix ignore parameters in dbpedia and msmarco-passage-ranking#761

Merged
jimczi merged 3 commits intoelastic:masterfrom
jimczi:track_total_hits_option
Mar 21, 2025
Merged

Fix ignore parameters in dbpedia and msmarco-passage-ranking#761
jimczi merged 3 commits intoelastic:masterfrom
jimczi:track_total_hits_option

Conversation

@jimczi
Copy link
Contributor

@jimczi jimczi commented Mar 20, 2025

The size and track_total_hits params are ignored in the dbpedia and msmarco-passage-ranking tracks. This change fixes the issue by providing the params as part of the search request body.

The `size` and `track_total_hits` params are ignored in the dbpedia and msmarco-passage-ranking tracks.
This change fixes the issue by providing the params as part of the search request body.
@jimczi jimczi added the bug label Mar 20, 2025
@jimczi jimczi requested a review from kderusso March 20, 2025 17:42
Copy link
Member

@kderusso kderusso left a comment

Choose a reason for hiding this comment

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

Thanks for fixing!

@jimczi jimczi merged commit ecf899a into elastic:master Mar 21, 2025
13 checks passed
@jimczi jimczi deleted the track_total_hits_option branch March 21, 2025 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants