Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

add yarn max allocation per heatbeat #2365

Merged
merged 2 commits into from Mar 21, 2019

Conversation

mzmssg
Copy link
Member

@mzmssg mzmssg commented Mar 21, 2019

adapt this patch to 2.9.0 to fix #2364 : https://issues.apache.org/jira/browse/YARN-8896

@coveralls
Copy link

Coverage Status

Coverage remained the same at 52.647% when pulling e037ee5 on zimiao/add_maxallocation_perheatbeat into 0f80208 on master.

@coveralls
Copy link

coveralls commented Mar 21, 2019

Coverage Status

Coverage remained the same at 52.647% when pulling 6c87e01 on zimiao/add_maxallocation_perheatbeat into 0f80208 on master.

Copy link
Contributor

@fanyangCS fanyangCS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you confirm it is indeed due to this bug?

@mzmssg
Copy link
Member Author

mzmssg commented Mar 21, 2019

@fanyangCS
Can't reproduce, but our scheduler get stuck in the same loop, so this workaround should work.

@@ -32,6 +32,8 @@ git checkout branch-2.9.0
git apply /hadoop-2.9.0.gpu-port.patch
git apply /HDFS-13773.patch
git apply /docker-executor.patch
# to avoid potential endless loop, refer to https://issues.apache.org/jira/browse/YARN-8513?page=com.atlassian.jira.plugin.system.issuetabpanels%3Aall-tabpanel
git apply /12944563.patch
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not use the orignal Patch name, i.e. YARN-8896.patch

Copy link
Member Author

@mzmssg mzmssg Mar 21, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok

@mzmssg mzmssg merged commit 0226bb3 into master Mar 21, 2019
mzmssg added a commit that referenced this pull request Mar 21, 2019
* add max allocation per heatbeat

* reneme patch for better readibility
mzmssg added a commit that referenced this pull request Mar 21, 2019
* add max allocation per heatbeat

* reneme patch for better readibility
@mzmssg mzmssg deleted the zimiao/add_maxallocation_perheatbeat branch March 28, 2019 08:09
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.

Resource manager get stuck
4 participants