You are viewing a plain text version of this content. The canonical link for it is here.
Posted to yarn-issues@hadoop.apache.org by "Gergely Pollák (Jira)" <ji...@apache.org> on 2021/08/04 13:29:00 UTC
[jira] [Resolved] (YARN-10355) Refactor NM
ContainerLaunch.java#orderEnvByDependencies
[ https://issues.apache.org/jira/browse/YARN-10355?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Gergely Pollák resolved YARN-10355.
-----------------------------------
Hadoop Flags: Reviewed
Target Version/s: 3.4.0
Resolution: Fixed
> Refactor NM ContainerLaunch.java#orderEnvByDependencies
> -------------------------------------------------------
>
> Key: YARN-10355
> URL: https://issues.apache.org/jira/browse/YARN-10355
> Project: Hadoop YARN
> Issue Type: Improvement
> Components: yarn
> Reporter: Benjamin Teke
> Assignee: Tamas Domok
> Priority: Minor
> Labels: pull-request-available
> Time Spent: 5h 20m
> Remaining Estimate: 0h
>
> The {{org.apache.hadoop.yarn.server.nodemanager.containermanager.launcher.ContainerLaunch#orderEnvByDependencies}} and it's helper method \{{getEnvDependencies }}(together with the overrides) is hard to read. Some improvements could be made:
> * use Pattern matching in the overrides of getEnvDependencies instead of iterating through the environmental variable strings char by char
> * the unit tests contains a lot of repeated code and generally the test methods are long - they could be separated into different setup/helper and assertion methods
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: yarn-issues-help@hadoop.apache.org