-
Notifications
You must be signed in to change notification settings - Fork 409
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
ThreadSanitizer reports data race in DeltaIndexManager::refreshRef. #4405
Labels
affects-5.0
affects-5.1
affects-5.2
affects-5.3
affects-5.4
affects-6.0
component/storage
severity/major
type/bug
The issue is confirmed as a bug.
Comments
12 tasks
ti-chi-bot
pushed a commit
that referenced
this issue
Mar 28, 2022
This was referenced Mar 28, 2022
This was referenced Mar 28, 2022
ti-chi-bot
added a commit
that referenced
this issue
Mar 28, 2022
12 tasks
ti-chi-bot
pushed a commit
that referenced
this issue
Apr 15, 2022
This was referenced Apr 27, 2022
ti-chi-bot
added a commit
that referenced
this issue
Apr 27, 2022
ti-chi-bot
added a commit
that referenced
this issue
Jun 15, 2022
ti-chi-bot
added a commit
that referenced
this issue
Jun 15, 2022
ti-chi-bot
added a commit
that referenced
this issue
Jun 16, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
affects-5.0
affects-5.1
affects-5.2
affects-5.3
affects-5.4
affects-6.0
component/storage
severity/major
type/bug
The issue is confirmed as a bug.
Bug Report
ThreadSanitizer reports data race in
DeltaIndexManager::refreshRef
.Following the code, it seems that
current_size
is not thread-safe?Please answer these questions before submitting your issue. Thanks!
1. Minimal reproduce step (Required)
Compile gtests_dbms in ThreadSanitizer mode and run it.
2. What did you expect to see? (Required)
No data race.
3. What did you see instead (Required)
Some data race happen.
4. What is your TiFlash version? (Required)
master
The text was updated successfully, but these errors were encountered: