Skip to content

[7.14] Introduce separate track for SQL (#223)#225

Merged
Luegg merged 1 commit intoelastic:7.14from
Luegg:backport/7.14/pr-223
Dec 6, 2021
Merged

[7.14] Introduce separate track for SQL (#223)#225
Luegg merged 1 commit intoelastic:7.14from
Luegg:backport/7.14/pr-223

Conversation

@Luegg
Copy link
Copy Markdown
Contributor

@Luegg Luegg commented Dec 6, 2021

Backports the following commits to 7.14:

Extracts the SQL challenge into its own track to make it easier to have separate dashboards for the SQL tasks and to reduce coupling between NOAA and SQL benchmarks.

This PR also addresses some other issues that have been identified in the meantime:

- Force-merges the index into a single segment to reduce variability between runs
- Adds the `wait-until-merges-finish` task to ensure merging finished before running the queries
- Introduces a `query_percentage` track parameter as a convenience to get faster results when doing experimentations (to be taken with a huge grain of salt of course)
- Ensures that `page_timeout` and `request_timeout` are configured correctly such that the track can be run on revisions with or without elastic/elasticsearch#79360.

Apart from these changes, the queries themselves have not been touched.
@Luegg Luegg added the backport This PR is a backport of some other PR label Dec 6, 2021
@Luegg Luegg merged commit 2ae1283 into elastic:7.14 Dec 6, 2021
@Luegg Luegg deleted the backport/7.14/pr-223 branch December 6, 2021 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of some other PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant