Skip to content

fix data race#736

Merged
disksing merged 4 commits intotikv:masterfrom
Smityz:fix-race
Mar 27, 2023
Merged

fix data race#736
disksing merged 4 commits intotikv:masterfrom
Smityz:fix-race

Conversation

@Smityz
Copy link
Contributor

@Smityz Smityz commented Mar 9, 2023

close #735

Smityz added 3 commits March 9, 2023 11:40
Signed-off-by: Smityz <smityz@qq.com>
Signed-off-by: Smityz <smityz@qq.com>
@Smityz
Copy link
Contributor Author

Smityz commented Mar 21, 2023

@disksing Hi master, could you please help me to review this fix? It is only related to unit testing.

@disksing disksing merged commit bb350d6 into tikv:master Mar 27, 2023
you06 pushed a commit to you06/client-go that referenced this pull request Jun 27, 2023
Signed-off-by: Smityz <smityz@qq.com>
Co-authored-by: disksing <i@disksing.com>
you06 pushed a commit to you06/client-go that referenced this pull request Jun 27, 2023
Signed-off-by: Smityz <smityz@qq.com>
Co-authored-by: disksing <i@disksing.com>
Signed-off-by: you06 <you1474600@gmail.com>
cfzjywxk pushed a commit that referenced this pull request Jun 28, 2023
#846)

* reload region cache when store is resolved from invalid

Signed-off-by: you06 <you1474600@gmail.com>

* reload region once

Signed-off-by: you06 <you1474600@gmail.com>

* build in go1.18

Signed-off-by: you06 <you1474600@gmail.com>

* build in go1.18

Signed-off-by: you06 <you1474600@gmail.com>

* handle region reload in resolve goroutine

Signed-off-by: you06 <you1474600@gmail.com>

* retest

Signed-off-by: you06 <you1474600@gmail.com>

* fix data race (#736)

Signed-off-by: Smityz <smityz@qq.com>
Co-authored-by: disksing <i@disksing.com>
Signed-off-by: you06 <you1474600@gmail.com>

* build with go 1.18

Signed-off-by: you06 <you1474600@gmail.com>

* fix integration test (#673)

Signed-off-by: disksing <i@disksing.com>
Signed-off-by: you06 <you1474600@gmail.com>

* Update internal/locate/region_cache.go

Co-authored-by: crazycs <crazycs520@gmail.com>
Signed-off-by: you06 <you1474600@gmail.com>

* address comment

Signed-off-by: you06 <you1474600@gmail.com>

* address comment

Signed-off-by: you06 <you1474600@gmail.com>

---------

Signed-off-by: you06 <you1474600@gmail.com>
Signed-off-by: Smityz <smityz@qq.com>
Signed-off-by: disksing <i@disksing.com>
Co-authored-by: Smilencer <smityz@qq.com>
Co-authored-by: disksing <i@disksing.com>
Co-authored-by: crazycs <crazycs520@gmail.com>
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.

race test failed

2 participants