Skip to content
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

Add concurrent control and test to verifier #17606

Merged
merged 1 commit into from
Apr 9, 2022

Conversation

aweisberg
Copy link
Contributor

@aweisberg aweisberg commented Apr 7, 2022

Test plan - Added unit tests

In some cases it is useful to run control and test queries concurrently. It can ensure long running queries are run at roughly the same time of day so the ambient load and queuing is the same, and it can also allow the overall latency of a test run to be lower.

== RELEASE NOTES ==

Verifier Changes
* Add support to verifier for running control and test queries concurrently through config `concurrent-control-and-test` and test property.

@aweisberg aweisberg force-pushed the verifier-concurrent branch 2 times, most recently from a8ffb60 to ce4cf6d Compare April 8, 2022 14:40
@aweisberg aweisberg force-pushed the verifier-concurrent branch from ce4cf6d to 67acdeb Compare April 8, 2022 15:23
@aweisberg aweisberg marked this pull request as ready for review April 8, 2022 15:37
@aweisberg aweisberg marked this pull request as draft April 8, 2022 16:02
@aweisberg aweisberg marked this pull request as ready for review April 8, 2022 17:56
@highker highker merged commit 7193625 into prestodb:master Apr 9, 2022
@mshang816 mshang816 mentioned this pull request May 17, 2022
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants