[SPARK-3476] Remove outdated memory checks in Yarn

[andrewor14]

See description in JIRA.

[AmplabJenkins]

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/20772/

[andrewor14]

Man, test this please

[AmplabJenkins]

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/20773/

[andrewor14]

Tested this on yarn cluster. Tests are failing because of git exception. Let's wait a little before retesting this.

[sryza]

The changes conceptually look good to me.

[tgravescs]

Yeah changes look good if Jenkins passes. We now have a map and for loop a single instance so we probably could condense it but not a big deal.

[SparkQA]

QA tests have started for PR 2528 at commit c5400cd.

• This patch merges cleanly.

[SparkQA]

QA tests have finished for PR 2528 at commit c5400cd.

• This patch passes unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/20813/

[andrewor14]

Yup, I have simplified it in my latest commit. Does this LGTYG?

[tgravescs]

yep, looks good. Thanks!

[andrewor14]

Ok, I'm merging this.