OpenHFT / Chronicle-Map

Replicate your Key Value Store across your network, with consistency, persistance and performance.
http://chronicle.software/products/chronicle-map/
Apache License 2.0
2.77k stars 469 forks source link

Concurrency bug fixes using Facebook infer RacerD static analysis tool #497

Closed yokurang closed 10 months ago

yokurang commented 11 months ago

Here is the static analysis report before my changes:

Pasted Graphic

Here is the static analysis report after my changes:

Untitled 15

They are simple changes using only synchronisation blocks. Further fine-graining might be necessary to ensure efficient concurrency, but I believe this is a good start to some read/write errors in some parts of the concurrent code.

P.S: I am using the RacerD static analysis tool to detect data races

I would appreciate any feedback / reviews. Thank you.