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

Improve integration suite performance #411

Merged
merged 5 commits into from
Jun 21, 2023

Conversation

sergiyvamz
Copy link
Contributor

Summary

Improve integration suite performance by creating required test DB clusters in advance.

Additional Reviewers

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sergiyvamz sergiyvamz force-pushed the perf-improvements-for-integration-suite branch 2 times, most recently from 7c156a4 to 069fb54 Compare June 5, 2023 16:24
@sergiyvamz sergiyvamz force-pushed the perf-improvements-for-integration-suite branch 3 times, most recently from 5730fc4 to dbf12ba Compare June 20, 2023 00:12
@sergiyvamz sergiyvamz force-pushed the perf-improvements-for-integration-suite branch from dbf12ba to 2775a8a Compare June 20, 2023 00:14
@sergiyvamz sergiyvamz changed the title [WIP] Improve integration suite performance Improve integration suite performance Jun 20, 2023
@@ -363,7 +363,7 @@ public void testServerFailoverWithIdleConnections() throws SQLException, Interru
conn));

// Sleep for a second to allow daemon threads to finish running.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
// Sleep for a second to allow daemon threads to finish running.
// Sleep for 10 seconds to allow daemon threads to finish running.

@sergiyvamz sergiyvamz merged commit 08ffbdd into main Jun 21, 2023
@sergiyvamz sergiyvamz deleted the perf-improvements-for-integration-suite branch June 21, 2023 17:31
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.

3 participants