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

roachtest: create a workload that tests latencies of repeated connection attempts #59394

Closed
rafiss opened this issue Jan 25, 2021 · 0 comments · Fixed by #62166
Closed

roachtest: create a workload that tests latencies of repeated connection attempts #59394

rafiss opened this issue Jan 25, 2021 · 0 comments · Fixed by #62166
Assignees
Labels
A-testing Testing tools and infrastructure C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) O-postmortem Originated from a Postmortem action item.

Comments

@rafiss
Copy link
Collaborator

rafiss commented Jan 25, 2021

The bug in #58671 caused the latency of establishing a secure connection to a multi-region cluster for a non-root user to dramatically increase. None of the roachtest workloads really test this, since they all use a connection pool where the connections are created once in the beginning, then reused for the rest of the test.

Create a test which is specifically for testing connection latency in a CockroachDB cluster. The test should report its results in roachperf.crdb.dev/ so that we can track performance changes over time.

Let's also just spot-check the performance of 21.1 compared to the latest 20.2 release.

@rafiss rafiss added C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) A-testing Testing tools and infrastructure labels Jan 25, 2021
@rafiss rafiss added the O-postmortem Originated from a Postmortem action item. label Jan 28, 2021
@craig craig bot closed this as completed in 1764c5f Apr 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testing Testing tools and infrastructure C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) O-postmortem Originated from a Postmortem action item.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants