You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@celeborn.apache.org by "AngersZhuuuu (via GitHub)" <gi...@apache.org> on 2024/02/26 10:21:20 UTC
[PR] [CELEBORN-1291]Master crashed causing by huge app level worker consumption info [incubator-celeborn]
AngersZhuuuu opened a new pull request, #2330:
URL: https://github.com/apache/incubator-celeborn/pull/2330
### What changes were proposed in this pull request?
When I upgrade to 0.4.0 and backport the app level consumption pr.
WorkerInfo consumption contains very huge information.
Since we enable debug level info for master, causing master print slots info very huge and stuck.
This pr fix this issue
### Why are the changes needed?
Fix bug
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@celeborn.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CELEBORN-1291] Master crashed causing by huge app level worker consumption info [incubator-celeborn]
Posted by "AngersZhuuuu (via GitHub)" <gi...@apache.org>.
AngersZhuuuu closed pull request #2330: [CELEBORN-1291] Master crashed causing by huge app level worker consumption info
URL: https://github.com/apache/incubator-celeborn/pull/2330
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@celeborn.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CELEBORN-1291] Master crashed causing by huge app level worker consumption info [incubator-celeborn]
Posted by "codecov[bot] (via GitHub)" <gi...@apache.org>.
codecov[bot] commented on PR #2330:
URL: https://github.com/apache/incubator-celeborn/pull/2330#issuecomment-1963859119
## [Codecov](https://app.codecov.io/gh/apache/incubator-celeborn/pull/2330?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
All modified and coverable lines are covered by tests :white_check_mark:
> Project coverage is 48.71%. Comparing base [(`7b0211e`)](https://app.codecov.io/gh/apache/incubator-celeborn/commit/7b0211e345751edb1e66e2760d9cae4d852adffc?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) to head [(`59eae27`)](https://app.codecov.io/gh/apache/incubator-celeborn/pull/2330?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
> Report is 1 commits behind head on main.
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## main #2330 +/- ##
==========================================
- Coverage 48.83% 48.71% -0.12%
==========================================
Files 207 207
Lines 12939 12944 +5
Branches 1111 1113 +2
==========================================
- Hits 6318 6304 -14
- Misses 6218 6235 +17
- Partials 403 405 +2
```
</details>
[:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/apache/incubator-celeborn/pull/2330?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
:loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@celeborn.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org