Add thread sanitizer to test

[NobodyXu]

This PR adds thread sanitizer to github workflow CI.

Signed-off-by: Jiahao XU Jiahao_XU@outlook.com

[vorner]

Also, as mentioned in the linked ticket… I'm hesitant to merge it right now until we know if the reported failure is a false positive or not and until there's a solution. But once once that is solved, I'm happy to have it.

[vorner]

Nice that the MacOS one works. But the UBuntu one would keep the CI red all the time, so it's not particularly useful.

Do you agree if I cherry-pick that one and leave the rest? Or do you want to create a cleaned-up branch with that one yourself?

[NobodyXu]

Nice that the MacOS one works. But the UBuntu one would keep the CI red all the time, so it's not particularly useful.

Do you agree if I cherry-pick that one and leave the rest? Or do you want to create a cleaned-up branch with that one yourself?

I will remove the ubuntu thread sanitizer test since it is likely to be just false positive and really isn't helpful.

[NobodyXu]

@vorner I've removed the Ubuntu thread sanitizer and rebased the branch again vorner:master.

All the workflows succeeded.

[codecov-commenter]

Codecov Report

Merging #72 (7c8398f) into master (f06b051) will increase coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #72      +/-   ##
==========================================
+ Coverage   86.25%   86.34%   +0.09%     
==========================================
  Files          18       18              
  Lines        1091     1091              
==========================================
+ Hits          941      942       +1     
+ Misses        150      149       -1     

Impacted Files	Coverage Δ
src/strategy/hybrid.rs	93.84% <0.00%> (+1.53%)	⬆️