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

Test thread safety during CI #2513

Closed
reyang opened this issue Oct 26, 2021 · 0 comments · Fixed by #4879
Closed

Test thread safety during CI #2513

reyang opened this issue Oct 26, 2021 · 0 comments · Fixed by #4879
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Good for taking. Extra help will be provided by maintainers infra Infra work - CI/CD, code coverage, linters

Comments

@reyang
Copy link
Member

reyang commented Oct 26, 2021

Feature Request

Is your feature request related to a problem?

While working on some of the components (e.g. #1085, #1086, #2477, #2505, #2506) that deal with concurrency/threads, I found a need to have better tooling support.

Describe the solution you'd like:

We probably should explore something like LLVM tsan.

Here are some options available:

Describe alternatives you've considered.

Currently we're kind of relying on code review and some stress test.

@reyang reyang added enhancement New feature or request help wanted Good for taking. Extra help will be provided by maintainers good first issue Good for newcomers infra Infra work - CI/CD, code coverage, linters labels Oct 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Good for taking. Extra help will be provided by maintainers infra Infra work - CI/CD, code coverage, linters
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant