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

Make recording package concurrency safe #18763

Closed
chlowell opened this issue Jul 29, 2022 · 0 comments · Fixed by #19610
Closed

Make recording package concurrency safe #18763

chlowell opened this issue Jul 29, 2022 · 0 comments · Fixed by #19610
Labels
MQ This issue is part of a "milestone of quality" initiative. test-enhancement

Comments

@chlowell
Copy link
Member

Live tests would run significantly faster if they executed test cases in parallel. That isn't possible today because our recording infrastructure isn't concurrency safe. Making it so would require at least synchronizing testSuite access (both reads and writes):

var testSuite = map[string]recordedTest{}

@chlowell chlowell added test-enhancement MQ This issue is part of a "milestone of quality" initiative. labels Jul 29, 2022
@chlowell chlowell added this to the Backlog milestone Jul 29, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Apr 11, 2023
@RickWinter RickWinter removed this from the Backlog milestone Dec 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
MQ This issue is part of a "milestone of quality" initiative. test-enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants