Dustin Cote updated YARN-2369:
    Attachment: YARN-2369-2.patch

New patch has a configurable whitelist for variables with append enabled as 
yarn.application.variables.with.append.  The existing unit tests are passing 
from TestMRApps.  Next up, a test to try to append to a variable not on the 
default white list and verify it gets replaced instead of appended to.  
[~jlowe], any thoughts on things that are missing from this latest patch or 
problems with the design?

> Environment variable handling assumes values should be appended
> ---------------------------------------------------------------
>                 Key: YARN-2369
>                 URL: https://issues.apache.org/jira/browse/YARN-2369
>             Project: Hadoop YARN
>          Issue Type: Bug
>    Affects Versions: 2.2.0
>            Reporter: Jason Lowe
>            Assignee: Dustin Cote
>         Attachments: YARN-2369-1.patch, YARN-2369-2.patch
> When processing environment variables for a container context the code 
> assumes that the value should be appended to any pre-existing value in the 
> environment.  This may be desired behavior for handling path-like environment 
> variables such as PATH, LD_LIBRARY_PATH, CLASSPATH, etc. but it is a 
> non-intuitive and harmful way to handle any variable that does not have 
> path-like semantics.

This message was sent by Atlassian JIRA

Reply via email to