Improve: Passing workerId to WorkerStat & Skip wait worker close if master executes failed

coderzc commented 9 months ago

Main Changes

Passing workerId to WorkerStat.
If master executes failed, skip wait worker close when close master.

Verifying these changes

[x] Trivial rework / code cleanup without any test coverage. (No Need)
[ ] Already covered by existing tests, such as (please modify tests here).
[ ] Need tests and can be verified as follows.

Does this PR potentially affect the following parts?

[x] Nope
[ ] Dependencies (add/update license info)
[ ] Modify configurations
[ ] The public API
[ ] Other affects (typed here)

Documentation Status

[ ] Doc - TODO
[ ] Doc - Done
[x] Doc - No Need

imbajin commented 9 months ago

CI error refer here: https://github.com/apache/incubator-hugegraph-computer/pull/286/commits/adf4c43100c71456d4a95b9f473efdf0c6c57f86#diff-ddf622d3d17b1ad33510d45f00504beb293ebefe76b3977511ae7033676628c0

codecov[bot] commented 9 months ago

Codecov Report

Attention: 8 lines in your changes are missing coverage. Please review.

Comparison is base (2be0c28) 85.06% compared to head (37a4242) 85.03%.

Files	Patch %	Lines
.../hugegraph/computer/core/master/MasterService.java	84.00%	5 Missing and 3 partials :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #292 +/- ## ============================================ - Coverage 85.06% 85.03% -0.04% + Complexity 3297 3296 -1 ============================================ Files 349 349 Lines 12478 12485 +7 Branches 1129 1130 +1 ============================================ + Hits 10615 10616 +1 - Misses 1325 1329 +4 - Partials 538 540 +2 ```

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

apache / incubator-hugegraph-computer