fix: subscriber data race

[jim3ma]

Signed-off-by: Jim Ma [email protected]

Description

Related Issue

Motivation and Context

Screenshots (if appropriate)

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Documentation Update (if none of the other choices apply)

Checklist

• Code compiles correctly.
• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[codecov-commenter]

Codecov Report

Merging #1169 (df0c8b4) into main (59dad99) will increase coverage by 0.11%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main    #1169      +/-   ##
==========================================
+ Coverage   46.07%   46.18%   +0.11%     
==========================================
  Files         132      132              
  Lines       10238    10239       +1     
==========================================
+ Hits         4717     4729      +12     
+ Misses       4977     4963      -14     
- Partials      544      547       +3     

Flag	Coverage Δ
Object-compatibility-e2etests	?
e2etests	?
unittests	46.18% <50.00%> (+0.11%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
client/daemon/rpcserver/subscriber.go	59.37% <50.00%> (+5.69%)	⬆️
client/daemon/peer/peertask_conductor.go	57.36% <0.00%> (+0.35%)	⬆️
scheduler/service/service.go	87.31% <0.00%> (+0.86%)	⬆️

[gaius-qi]

LGTM

[gaius-qi]

LGTM

[gaius-qi]

LGTM