Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

[Launcher]: Add more info into SummarizedFrameworkInfo #2435

Merged
merged 3 commits into from
Mar 28, 2019

Conversation

yqwang-ms
Copy link
Member

  1. Add more info into SummarizedFrameworkInfo
    Add TotalGpuCount and TotalTaskCount into job list #2228

  2. Skip to stop ZKStore in case it is still using
    Graceful stop for tasks and OOM conditions handling #2408

@yqwang-ms yqwang-ms requested review from fanyangCS and mzmssg March 28, 2019 06:13
@yqwang-ms
Copy link
Member Author

Example:
image

@mzmssg
Copy link
Member

mzmssg commented Mar 28, 2019

We only provide total GPU number, without cpu or memory?

@yqwang-ms
Copy link
Member Author

Good point, let me added it.

@mzmssg
Copy link
Member

mzmssg commented Mar 28, 2019

@yqwang-ms
Another question, should it count AM resource

@coveralls
Copy link

coveralls commented Mar 28, 2019

Coverage Status

Coverage increased (+0.3%) to 52.936% when pulling 1fb52a9 on yqwang/launcher-dev into deeee3f on master.

@yqwang-ms
Copy link
Member Author

It seems not easy to judge this. So, let's only add current fields, the memory and cpu is TBD in future release.

@yqwang-ms yqwang-ms merged commit a7525dc into master Mar 28, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants