search

intelligent-machine-learning / dlrover

DLRover: An Automatic Distributed Deep Learning System
Other
1.22k stars 153 forks source link

optimize heartbeat collect #1264

Closed BalaBalaYi closed 2 weeks ago

BalaBalaYi commented 2 weeks ago

What changes were proposed in this pull request?

Avoid key error.

Why are the changes needed?

To avoid exception when master recovered.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

UT.

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 80.56%. Comparing base (c72d7d0) to head (162b643). Report is 3 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1264 +/- ## ========================================== - Coverage 80.58% 80.56% -0.02% ========================================== Files 218 218 Lines 19982 20009 +27 ========================================== + Hits 16102 16121 +19 - Misses 3880 3888 +8 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.