Skip to content

Fix versioning for tests cases using a randomly generated rank builder#95514

Merged
jdconrad merged 2 commits intoelastic:mainfrom
jdconrad:rrffailure
Apr 24, 2023
Merged

Fix versioning for tests cases using a randomly generated rank builder#95514
jdconrad merged 2 commits intoelastic:mainfrom
jdconrad:rrffailure

Conversation

@jdconrad
Copy link
Contributor

Certain tests generate a random search request. These search requests aren't versioned, so in certain tests if a rank builder is set, the versions must be appropriately gated or the rank builder must be set to null. This change fixes the issues for testForceUnsupportedSynthetic and testChannelVersion.

@jdconrad jdconrad added >bug :Search Relevance/Ranking Scoring, rescoring, rank evaluation. v8.8.0 labels Apr 24, 2023
@elasticsearchmachine elasticsearchmachine added the Team:Search Meta label for search team label Apr 24, 2023
@elasticsearchmachine
Copy link
Collaborator

Hi @jdconrad, I've created a changelog YAML for you.

@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

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

LGTM

@jdconrad
Copy link
Contributor Author

@rjernst Thanks for the review!

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

Labels

>bug :Search Relevance/Ranking Scoring, rescoring, rank evaluation. Team:Search Meta label for search team v8.8.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants